Skip to content

Import checkArgument statically#17811

Merged
martint merged 1 commit intotrinodb:masterfrom
martint:import
Jun 9, 2023
Merged

Import checkArgument statically#17811
martint merged 1 commit intotrinodb:masterfrom
martint:import

Conversation

@martint
Copy link
Copy Markdown
Member

@martint martint commented Jun 9, 2023

Release notes

(x) This is not user-visible or docs only and no release notes are required.

@cla-bot cla-bot bot added the cla-signed label Jun 9, 2023
@martint martint mentioned this pull request Jun 9, 2023
Merged
ebyhr
ebyhr approved these changes Jun 9, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@ebyhr ebyhr left a comment

Choose a reason for hiding this comment

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

#17774 added the package name in the following classes too:

  • core/trino-spi/src/main/java/io/trino/spi/function/table/Descriptor.java
  • core/trino-spi/src/main/java/io/trino/spi/function/table/TableFunctionAnalysis.java

@martint martint merged commit 664dc57 into trinodb:master Jun 9, 2023
@martint martint deleted the import branch June 9, 2023 13:17
@github-actions github-actions bot added this to the 420 milestone Jun 9, 2023
@colebow colebow mentioned this pull request Jun 21, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wendigo wendigo wendigo approved these changes

@ebyhr ebyhr ebyhr approved these changes

Assignees

No one assigned

Labels

cla-signed

Milestone

420

Development

Successfully merging this pull request may close these issues.

3 participants

@martint @wendigo @ebyhr