Skip to content

sql: make pg_description aware of builtin function descriptions#95294

Merged
craig[bot] merged 2 commits intocockroachdb:masterfrom
knz:20230116-builtin-desc
Jan 17, 2023
Merged

sql: make pg_description aware of builtin function descriptions#95294
craig[bot] merged 2 commits intocockroachdb:masterfrom
knz:20230116-builtin-desc

Conversation

@knz
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

knz commented Jan 17, 2023

Thanks to both!

bors r=rafiss,msirek

@craig
Copy link
Copy Markdown
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