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

Skip getTableHandle call in information_schema.columns with table filter #18730

Conversation

findepi
Copy link
Member

@findepi findepi commented Aug 18, 2023

When reading information_schema.columns with a filter on table name (either = or IN), the code was doing
getRedirectionAwareTableHandle call to check whether target table exists. However, the redirected tables are handled in doListTableColumns (also catching TABLE_REDIRECTION_ERROR), so the call inside calculatePrefixesWithTableName is redundant.

@cla-bot cla-bot bot added the cla-signed label Aug 18, 2023
When reading `information_schema.columns` with a filter on table name
(either `=` or `IN`), the code was doing
`getRedirectionAwareTableHandle` call to check whether target table
exists. However, the redirected tables are handled in
`doListTableColumns` (also catching `TABLE_REDIRECTION_ERROR`), so the
call inside `calculatePrefixesWithTableName` is redundant.
@findepi findepi force-pushed the findepi/skip-gettablehandle-call-in-information-schema-columns-with-table-filter-2580aa branch from ae2a019 to 25ab369 Compare August 18, 2023 12:25
@findepi findepi requested review from martint, ebyhr, phd3 and kokosing August 29, 2023 12:57
@findepi findepi merged commit c9fa5ff into trinodb:master Aug 30, 2023
@findepi findepi deleted the findepi/skip-gettablehandle-call-in-information-schema-columns-with-table-filter-2580aa branch August 30, 2023 05:06
@findepi findepi added the no-release-notes This pull request does not require release notes entry label Aug 30, 2023
@github-actions github-actions bot added this to the 426 milestone Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed no-release-notes This pull request does not require release notes entry
Development

Successfully merging this pull request may close these issues.

3 participants