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.

Sign up for GitHub

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

Jump to bottom

sql: grant to all tables did not skip privileges on sequences #120685

Merged
merged 1 commit into from
Mar 19, 2024
Merged

sql: grant to all tables did not skip privileges on sequences #120685

merged 1 commit into from
Mar 19, 2024

Commits on Mar 19, 2024

  1. sql: grant to all tables did not skip privileges on sequences 

    Previously, grant privilege to all tables attempted to apply privileges
meant only for tables on to sequences. This could lead to validation
error preventing certain combinations like "GRANT BACKUP ON ALL
TABLES..." from working correctly. To address this, this patch adds
support for correctly propogating the object type and skipping over
unsupported privileges. When an unsupported privilege is encountered on
an object other then a sequence a warning is now logged.

Fixes: cockroachdb#117861

Release note (bug fix): GRANT <...> ON ALL TABLES could fail if
sequences existed and they did not support a privilege (for example
BACKUP).
    @fqazi
    fqazi committed Mar 19, 2024
    Configuration menu
    Copy the full SHA
    007005f View commit details
    Browse the repository at this point in the history