-
Notifications
You must be signed in to change notification settings - Fork 60
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
override get_columns_in_relation with driver api call #554
override get_columns_in_relation with driver api call #554
Conversation
This PR has been marked as Stale because it has been open with no activity as of late. If you would like the PR to remain open, please comment on the PR or else it will be closed in 7 days. |
Although we are closing this PR as stale, it can still be reopened to continue development. Just add a comment to notify the maintainers. |
This PR will be superceded by #899. @jiezhen-chen will be given contributor credit in that PR as the implementation is effectively the same as here. |
resolves dbt-labs/dbt-adapters#645
redshift__get_columns_relation currently is a hardcoded sql query that references the pg catalog. Changes in this PR leverage get_columns, a redshift_connector API method, to retrieve the same data.
Checklist