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: pg_proc should respect context db for udf #88167

Merged

Conversation

chengxiong-ruan
Copy link
Contributor

@chengxiong-ruan chengxiong-ruan commented Sep 19, 2022

pg_proc table should only return udf in the context db if it's not nil.

Release note: None
Release justification: low risk virtual table improvement.

@chengxiong-ruan chengxiong-ruan requested a review from a team September 19, 2022 19:59
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@chengxiong-ruan chengxiong-ruan changed the title <pkg>: <short description - lowercase, no final period> sql: pg_proc should respect context db for udf Sep 19, 2022
`pg_proc` table should only return udf in the context db
if it's not nil.

Release note: None
Release justification: low risk virtual table improvement.
@chengxiong-ruan chengxiong-ruan force-pushed the udf-pg-proc-respect-current-db branch from b8b7773 to 9edd63b Compare September 19, 2022 20:00
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.

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @chengxiong-ruan)

@chengxiong-ruan
Copy link
Contributor Author

TFTR!
bors r+

@craig
Copy link
Contributor

craig bot commented Sep 20, 2022

Build failed (retrying...):

@chengxiong-ruan
Copy link
Contributor Author

looks like flaky tests..
bors r+

@craig
Copy link
Contributor

craig bot commented Sep 20, 2022

Already running a review

@craig
Copy link
Contributor

craig bot commented Sep 20, 2022

Build succeeded:

@craig craig bot merged commit 5862e9b into cockroachdb:master Sep 20, 2022
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.

3 participants