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/sem/tree: remove duplicate tree.NewDOidWithName function #128987

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

mgartner
Copy link
Collaborator

@mgartner mgartner commented Aug 14, 2024

The tree.NewDOidWithName function has been removed. All previous
callers now use tree.NewDOidWithTypeAndName which is exactly the same.

Epic: None

Release note: None

The `tree.NewDOidWithName` function has been removed. All previous
callers now use `tree.NewDOidWithTypeAndName` which is exactly the same.

Release note: None
@mgartner mgartner requested a review from a team August 14, 2024 17:24
@mgartner mgartner requested review from a team as code owners August 14, 2024 17:24
@mgartner mgartner requested review from mw5h and removed request for a team August 14, 2024 17:24
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@mgartner
Copy link
Collaborator Author

TFTR!

bors r+

@craig
Copy link
Contributor

craig bot commented Aug 15, 2024

Build failed (retrying...):

@craig craig bot merged commit 7b3e097 into cockroachdb:master Aug 15, 2024
23 checks passed
@mgartner mgartner deleted the oid-simplify branch August 16, 2024 16:10
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