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: don't no-op SET SESSION AUTHORIZATION DEFAULT #86485

Merged
merged 3 commits into from
Aug 22, 2022
Merged

sql: don't no-op SET SESSION AUTHORIZATION DEFAULT #86485

merged 3 commits into from
Aug 22, 2022

Commits on Aug 20, 2022

  1. sql: don't no-op SET SESSION AUTHORIZATION DEFAULT 

    Release note (bug fix): Previously, SET SESSION AUTHORIZATION DEFAULT
would have no effect. Now, it causes the current role to be reset to the
original user who logged into the session.

Release justification: low risk bug fix to existing functionality
    @rafiss
    rafiss committed Aug 20, 2022
    Configuration menu
    Copy the full SHA
    42b67ff View commit details
    Browse the repository at this point in the history

Commits on Aug 21, 2022

  1. sql: refactor DISCARD so it runs during execution 

    Previously, it would run during planning, which is not correct, or at
least, makes it harder to reason about what's going on.

Release note: None

Release justification: low risk refactor
    @rafiss
    rafiss committed Aug 21, 2022
    Configuration menu
    Copy the full SHA
    04bf2f3 View commit details
    Browse the repository at this point in the history

  2. sql: consolidate logic for RESET ALL 

    Release note: None

Release justification: low risk refactor
    @rafiss
    rafiss committed Aug 21, 2022
    Configuration menu
    Copy the full SHA
    3887560 View commit details
    Browse the repository at this point in the history