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

sql: make pg_description aware of builtin function descriptions #95294

Merged
merged 2 commits into from
Jan 17, 2023

Conversation

knz
Copy link
Contributor

@knz knz commented Jan 16, 2023

Epic: CRDB-23454
Fixes #95292.
Needed for #88061.

First commit from #95289.

This also extends the completion rules to properly handle
functions in multiple namespaces.

Release note (bug fix): pg_catalog.pg_description and pg_catalog.obj_description() are now able to retrieve the descriptive help for built-in functions.

@knz knz changed the title sql: populate pg_proc.prokind sql: make pg_description aware of builtin function descriptions Jan 16, 2023
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@knz knz force-pushed the 20230116-builtin-desc branch from 55e4d6c to 93f9b44 Compare January 16, 2023 15:36
@knz knz marked this pull request as ready for review January 16, 2023 15:36
@knz knz requested review from a team and rafiss January 16, 2023 15:36
@knz knz force-pushed the 20230116-builtin-desc branch 2 times, most recently from 2fb14ab to 6977d89 Compare January 16, 2023 16:22
@knz knz requested a review from a team as a code owner January 16, 2023 16:22
@knz knz force-pushed the 20230116-builtin-desc branch from 6977d89 to 31e18d1 Compare January 16, 2023 20:54
@knz knz requested a review from a team as a code owner January 16, 2023 20:54
@knz knz requested a review from msirek January 16, 2023 20:54
@knz knz force-pushed the 20230116-builtin-desc branch 2 times, most recently from 5eb76f8 to 8ce5a10 Compare January 17, 2023 14:19
knz added 2 commits January 17, 2023 15:26
Release note (bug fix): The `prokind` column of `pg_catalog.pg_proc`
is now populated properly.
This also extends the completion rules to properly handle
functions in multiple namespaces.

Release note (bug fix): `pg_catalog.pg_description` and
`pg_catalog.obj_description()` are now able to retrieve the
descriptive help for built-in functions.
@knz knz force-pushed the 20230116-builtin-desc branch from 8ce5a10 to bd2953b Compare January 17, 2023 14:27
Copy link
Collaborator

@rafiss rafiss left a comment

Choose a reason for hiding this comment

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

lgtm!

Reviewed 1 of 3 files at r1, 10 of 10 files at r2, all commit messages.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @msirek)

Copy link
Contributor

@msirek msirek left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @knz)

@knz
Copy link
Contributor Author

knz commented Jan 17, 2023

Thanks to both!

bors r=rafiss,msirek

@craig
Copy link
Contributor

craig bot commented Jan 17, 2023

Build succeeded:

@craig craig bot merged commit 6fd5b04 into cockroachdb:master Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sql: unable to retrieve built-in function descriptions via pg_objdescription
4 participants