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

[Redshift Dv2] Handle svv_table_info permissions #33552

Closed
cynthiaxyin opened this issue Dec 16, 2023 · 1 comment · Fixed by #34186
Closed

[Redshift Dv2] Handle svv_table_info permissions #33552

cynthiaxyin opened this issue Dec 16, 2023 · 1 comment · Fixed by #34186
Assignees
Labels
team/destinations Destinations team's backlog

Comments

@cynthiaxyin
Copy link
Contributor

cynthiaxyin commented Dec 16, 2023

From Slack thread.

(paraphrased) V2 introduced a new "is your table empty" check, which relies on querying the svv_table_info view. we could modify that check to do some sort of select exists(...) query to avoid asking for more permissions, but presumably there’s a minor cost tradeoff there, especially wrt to upgrading.

Data point: beta user was OK with granting more permission to the Airbyte user (if we stick with this route, we should add this info to the docs)

@cynthiaxyin cynthiaxyin added the team/destinations Destinations team's backlog label Dec 16, 2023
@evantahler
Copy link
Contributor

Grooming:

  • Dv2 needs more permissions. This table is what we use to get the approx # rows in the table (how we can tell if the table exists w.o doing a read). This is equivalent to INFORMATION_SCHEMA (trading a potentially harder setup for faster syncs). We should:
    1. Add a read to CHECK command to make sure we can access. We should try a SELECT * FROM svv_table_info LIMIT 1 to check access.
    2. Add to docs that we need READ permission on this table.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team/destinations Destinations team's backlog
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants