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

Register has_database/schema_privilege functions under pg_catalog #15770

Merged
merged 1 commit into from Mar 27, 2024

Conversation

matriv
Copy link
Contributor

@matriv matriv commented Mar 27, 2024

Although those 2 functions were available in the search path, they were not registered under pg_catalog schema, as the should, and as a result pg compatible tools which my use full path (e.g.
pg_catalog.has_shema_privilege()) would get an Uknown Function error.

Fixes: #15749

Although those 2 functions were available in the search path, they were
not registered under `pg_catalog` schema, as the should, and as a result
pg compatible tools which my use full path (e.g.
`pg_catalog.has_shema_privilege()`) would get an `Uknown Function` error.

Fixes: #15749
@matriv matriv requested a review from romseygeek March 27, 2024 11:35
Copy link
Contributor

@romseygeek romseygeek left a comment

Choose a reason for hiding this comment

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

LGTM

@matriv matriv added v/5.6 ready-to-merge Let Mergify merge the PR once approved and checks pass labels Mar 27, 2024
@mergify mergify bot merged commit 0eb245b into master Mar 27, 2024
18 checks passed
@mergify mergify bot deleted the mt/pg_catalog_has_privs branch March 27, 2024 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Let Mergify merge the PR once approved and checks pass v/5.6
Projects
None yet
Development

Successfully merging this pull request may close these issues.

has_schema_privilege should be accessible under pg_catalog
2 participants