Skip to content
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

dbt is adding incorrectly cased values to the cache #1140

Closed
beckjake opened this issue Nov 16, 2018 · 0 comments
Closed

dbt is adding incorrectly cased values to the cache #1140

beckjake opened this issue Nov 16, 2018 · 0 comments
Labels
bug Something isn't working
Milestone

Comments

@beckjake
Copy link
Contributor

beckjake commented Nov 16, 2018

Issue

On snowflake, dbt is incorrectly adding lowercase and uppercase versions of model names to the cache somehow. In particular, models added via add after a materialization is successfully executed are potentially the wrong case.

Issue description

dbt run twice on a snowflake project with casing set defaults (I think?) and lowercase model filenames. If you dump the cache after, you'll see both sets of entries in the cache - correctly uppercased from the initial population and incorrectly uppercased from the add calls.

Results

The cache should only think there's one set of names. On snowflake this is benign because we don't have any relations to drop anyway, but on postgres with odd casing choices (uppercase models + quoting=False) one can imagine problems

System information

Any dbt with caching (0.12+), any OS, any python.

@drewbanin drewbanin added this to the Grace Kelly milestone Nov 16, 2018
@drewbanin drewbanin added the bug Something isn't working label Nov 16, 2018
beckjake added a commit that referenced this issue Nov 20, 2018
…y-issues

Fix run repeatability/caching issues (#1138, #1139, #1140)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants