Skip to content

sql: pg_proc should respect context db for udf#88167

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
chengxiong-ruan:udf-pg-proc-respect-current-db
Sep 20, 2022
Merged

sql: pg_proc should respect context db for udf#88167
craig[bot] merged 1 commit intocockroachdb:masterfrom
chengxiong-ruan:udf-pg-proc-respect-current-db

Conversation

@chengxiong-ruan
Copy link
Copy Markdown
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
Copy Markdown
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
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.

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

TFTR!
bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Sep 20, 2022

Build failed (retrying...):

@chengxiong-ruan
Copy link
Copy Markdown
Contributor Author

looks like flaky tests..
bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Sep 20, 2022

Already running a review

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