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

Add redirection awareness for DROP COLUMN task #11304

Merged
merged 1 commit into from
Mar 8, 2022

Conversation

findinpath
Copy link
Contributor

@findinpath findinpath commented Mar 3, 2022

Description

Is this change a fix, improvement, new feature, refactoring, or other?

This is a new feature.

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

This change affects the core query engine.

How would you describe this change to a non-technical end user or system administrator?

Add redirection awareness for the table name in DROP COLUMN statements.

Related issues, pull requests, and links

Documentation

(x) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

( ) No release notes entries required.
(x) Release notes entries required with the following suggested text:

# Main
* Add redirection awareness for `DROP COLUMN` task.

@findepi
Copy link
Member

findepi commented Mar 4, 2022

FYI, the CI is red.

@findinpath findinpath force-pushed the drop-column-redirection-awareness branch from e16ea24 to 5a34970 Compare March 4, 2022 20:31
@findinpath findinpath force-pushed the drop-column-redirection-awareness branch from 5a34970 to 06b19c5 Compare March 7, 2022 11:18
@phd3 phd3 merged commit bafb4ac into trinodb:master Mar 8, 2022
@github-actions github-actions bot added this to the 373 milestone Mar 8, 2022
@phd3 phd3 mentioned this pull request Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants