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

fix: Updates dynamic tables query parsing to correctly handle tables that include "as" in their name (#2329) #2416

Closed
wants to merge 1 commit into from

Conversation

dtiesling
Copy link

The Index in the query parsing is too greedy and grabs any string with the letter "as" in them. If a table has "as" then the query is never a match and the dynamic is recreated in every terraform plan.

Test Plan

I have not tested locally since I'm not comfortable running acceptance tests against our live snowflake account. Creating a draft PR in hopes that CI will run tests and you all can review and verify the change.

References

Attempts to fix #2329

@sfc-gh-asawicki
Copy link
Collaborator

Hey. It wouldn't work, e.g. because you can have " AS " as a part of the comment. Also, the cut number was not adjusted (3 -> 5).

I am closing it and following up on it in #2421 PR.

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

Successfully merging this pull request may close these issues.

Dynamic tables are not parsing the query correctly and recreate resources on every apply
2 participants