-
Notifications
You must be signed in to change notification settings - Fork 59
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
Fixes #641 by using the svv_mv_info view which is queryable #647
Conversation
…ueryable in redshift-serverless
@dbeatty10 here's a wee little PR |
@jiezhen-chen any reason we shouldn't use |
In particular, is there a reason to use |
* Hopefully fixes #641 by using the svv_mv_info view which is queryable in redshift-serverless * Changelog entry * Explain rationale in a comment to reduce possibility of an accidental reversion --------- Co-authored-by: Doug Beatty <[email protected]> Co-authored-by: Doug Beatty <[email protected]> Co-authored-by: Mike Alfare <[email protected]> Co-authored-by: colin-rogers-dbt <[email protected]> (cherry picked from commit 058a3ff)
) * Hopefully fixes #641 by using the svv_mv_info view which is queryable in redshift-serverless * Changelog entry * Explain rationale in a comment to reduce possibility of an accidental reversion --------- Co-authored-by: Doug Beatty <[email protected]> Co-authored-by: Doug Beatty <[email protected]> Co-authored-by: Mike Alfare <[email protected]> Co-authored-by: colin-rogers-dbt <[email protected]> (cherry picked from commit 058a3ff) Co-authored-by: reptillicus <[email protected]>
* Hopefully fixes #641 by using the svv_mv_info view which is queryable in redshift-serverless * Changelog entry * Explain rationale in a comment to reduce possibility of an accidental reversion --------- Co-authored-by: Doug Beatty <[email protected]> Co-authored-by: Doug Beatty <[email protected]> Co-authored-by: Mike Alfare <[email protected]> Co-authored-by: colin-rogers-dbt <[email protected]> (cherry picked from commit 058a3ff)
…is queryable (#691) * Fixes #641 by using the svv_mv_info view which is queryable (#647) * Hopefully fixes #641 by using the svv_mv_info view which is queryable in redshift-serverless * Changelog entry * Explain rationale in a comment to reduce possibility of an accidental reversion --------- Co-authored-by: Doug Beatty <[email protected]> Co-authored-by: Doug Beatty <[email protected]> Co-authored-by: Mike Alfare <[email protected]> Co-authored-by: colin-rogers-dbt <[email protected]> (cherry picked from commit 058a3ff) * manually apply backported updates of #647 to macro that was moved between 1.6 and 1.7 --------- Co-authored-by: reptillicus <[email protected]>
resolves #641
Problem
When doing a dbt run on a redshift serverless cluster, permission denied errors will happen when dbt attempts to query the svt_mv_info table. This is due to that table not being queryable by any user in redshift serverless as described here.
Solution
This can be remedied by a custom macro for redshift__describe_materialized_view, which references the svv_mv_info view instead of the svt_mv_info table as shown below
Checklist