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: add test for builtin that belongs to information_schema #96468

Merged
merged 1 commit into from
Feb 8, 2023

Conversation

rafiss
Copy link
Collaborator

@rafiss rafiss commented Feb 2, 2023

Epic: None

This tests the change in 3f50d81

Release note: None

@rafiss rafiss requested a review from a team February 2, 2023 21:27
@blathers-crl
Copy link

blathers-crl bot commented Feb 2, 2023

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@rafiss rafiss force-pushed the qualify-info-schema branch from 285ad70 to d2f1703 Compare February 2, 2023 22:18
@rafiss rafiss requested a review from a team as a code owner February 2, 2023 22:18
@rafiss rafiss requested a review from ZhouXing19 February 4, 2023 00:36
Copy link
Collaborator

@ZhouXing19 ZhouXing19 left a comment

Choose a reason for hiding this comment

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

Seems that it has been fixed by 3f50d81 but we can still add the test for logic_test/pg_catalog

@rafiss rafiss force-pushed the qualify-info-schema branch from d2f1703 to 0121d6d Compare February 7, 2023 21:26
@rafiss
Copy link
Collaborator Author

rafiss commented Feb 7, 2023

tftr! i didn't see that commit go in earlier

bors r+

@rafiss rafiss changed the title sql: make builtins that belong to information_schema reported correctly sql: add test for builtin that belongs to information_schema Feb 7, 2023
@craig
Copy link
Contributor

craig bot commented Feb 7, 2023

Build failed (retrying...):

@rafiss
Copy link
Collaborator Author

rafiss commented Feb 7, 2023

bors r-

@craig
Copy link
Contributor

craig bot commented Feb 7, 2023

Canceled.

This tests the change in 3f50d81

Release note: None
@rafiss rafiss force-pushed the qualify-info-schema branch from 0121d6d to cba72da Compare February 8, 2023 03:05
@rafiss
Copy link
Collaborator Author

rafiss commented Feb 8, 2023

bors r+

@craig
Copy link
Contributor

craig bot commented Feb 8, 2023

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Feb 8, 2023

Build succeeded:

@craig craig bot merged commit d4f63d7 into cockroachdb:master Feb 8, 2023
@rafiss rafiss deleted the qualify-info-schema branch February 13, 2023 19:36
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