Skip to content

Remove UserHasSuperAccess as Doltgres now has built-in authorization for Dolt stored procedures #10159

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
Open
elianddb wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Remove UserHasSuperAccess as Doltgres now has built-in authorization for Dolt stored procedures #10159

elianddb wants to merge 4 commits into from

Conversation

@elianddb
Copy link
Contributor

@elianddb elianddb commented Dec 3, 2025 •
edited
Loading

No description provided.

@coffeegoddd
Copy link
Contributor

coffeegoddd commented Dec 3, 2025

@elianddb DOLT

comparing_percentages
100.000000 to 100.000000
version result total
eaa3b57 ok 5937471
version total_tests
eaa3b57 5937471
correctness_percentage
100.0

@coffeegoddd
Copy link
Contributor

coffeegoddd commented Dec 3, 2025

@elianddb DOLT

comparing_percentages
100.000000 to 100.000000
version result total
7342810 ok 5937471
version total_tests
7342810 5937471
correctness_percentage
100.0

@elianddb elianddb changed the title dolthub/dolt#7628: Fix Doltgres panic on no init() of UserHasSuperAccess for dolt_backup dolthub/dolt#7628: Remove UserHasSuperAccess as Doltgres now has built-in authorization for Dolt stored procedures Dec 4, 2025
@elianddb elianddb changed the title dolthub/dolt#7628: Remove UserHasSuperAccess as Doltgres now has built-in authorization for Dolt stored procedures Remove UserHasSuperAccess as Doltgres now has built-in authorization for Dolt stored procedures Dec 4, 2025
@elianddb elianddb mentioned this pull request Dec 4, 2025
Open
@elianddb elianddb marked this pull request as ready for review December 4, 2025 02:32
Hydrocharged
Hydrocharged approved these changes Dec 4, 2025
View reviewed changes
Copy link
Contributor

@Hydrocharged Hydrocharged left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@coffeegoddd
Copy link
Contributor

coffeegoddd commented Dec 5, 2025

@elianddb DOLT

comparing_percentages
100.000000 to 100.000000
version result total
7045301 ok 5937471
version total_tests
7045301 5937471
correctness_percentage
100.0

@elianddb elianddb force-pushed the elian/7628c branch 3 times, most recently from dc2a2c7 to 3a422d8 Compare December 5, 2025 21:24
@coffeegoddd
Copy link
Contributor

coffeegoddd commented Dec 6, 2025

@elianddb DOLT

comparing_percentages
100.000000 to 100.000000
version result total
752db82 ok 5937471
version total_tests
752db82 5937471
correctness_percentage
100.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Hydrocharged Hydrocharged Hydrocharged approved these changes

Assignees

No one assigned

Labels

correctness_approved

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@elianddb @coffeegoddd @Hydrocharged