Skip to content

schema override for multi tenant support #953

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

bakoliasn
Copy link

Describe your changes

If the override environment variable is set, replaces the table prefix with override and prepends the original prefix to the table name.

How has this been tested?

integration_tests

Reviewer focus

Most changes are in the models and seeds, setting the schema and alias

Checklist before requesting a review

  • I have added at least one Github label to this PR (bug, enhancement, breaking change,...)
  • My code follows style guidelines
  • (New models) YAML files are categorized by sub folder and models listed in alphabetical order
  • (New models) I have added a config to each new model to enable it for claims and/or clinical data
  • (New models) I have added the variable tuva_last_run to the final output
  • (Optional) I have recorded a Loom to explain this PR

Package release checklist

  • I have updated dbt docs
  • I have updated the version number in the dbt_project.yml

@bakoliasn bakoliasn marked this pull request as ready for review June 30, 2025 16:39
@davidshimamoto
Copy link
Collaborator

@bakoliasn Did you mean to fork this and commit this to your fork?

@bakoliasn
Copy link
Author

@bakoliasn Did you mean to fork this and commit this to your fork?

Thanks for the heads up! Yes, the goal is to upstream this change so we don’t have to maintain a separate fork. Happy to adjust anything needed to align with your contribution guidelines — just let me know what you'd prefer.

@davidshimamoto
Copy link
Collaborator

The process should be:

  1. Start a discussion on our community slack around the problem you're trying to solve.
  2. Open an issue describing the problem. If it is something extensive, discuss with the team managing the open source.
  3. Do the work.

That being said, this is a rather extensive change with architecture choices that requires some thought. I'd like to start by asking if you can elaborate on the problem you're trying to solve here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 👀 Ready for Review
Development

Successfully merging this pull request may close these issues.

2 participants