Skip to content

Conversation

@gruuya
Copy link
Contributor

@gruuya gruuya commented Aug 17, 2023

Which issue does this PR close?

Closes #7315.

Rationale for this change

Add qualifiers to the filter fields of the UPDATE statement plan.

What changes are included in this PR?

See above

Are these changes tested?

Existing tests altered to accordingly.

Are there any user-facing changes?

The EXPLAIN will now show qualified fields for the filter node of the UPDATE plan.

@github-actions github-actions bot added the sql SQL Planner label Aug 17, 2023
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Makes sense to me -- thank you @gruuya

@alamb alamb merged commit f2c0100 into apache:main Aug 17, 2023
@gruuya gruuya deleted the qualify-update-filter-field branch August 17, 2023 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sql SQL Planner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Filter fields in the UPDATE plan are unqualified

2 participants