Skip to content

Comments

Report current operation when coercion fails#13628

Merged
findepi merged 1 commit intoapache:mainfrom
findepi:findepi/errctx
Dec 4, 2024
Merged

Report current operation when coercion fails#13628
findepi merged 1 commit intoapache:mainfrom
findepi:findepi/errctx

Conversation

@findepi
Copy link
Member

@findepi findepi commented Dec 3, 2024

This makes for a more actionable error message.

@github-actions github-actions bot added the logical-expr Logical plan and expressions label Dec 3, 2024
@github-actions github-actions bot added the optimizer Optimizer rules label Dec 3, 2024
@github-actions github-actions bot added the sqllogictest SQL Logic Tests (.slt) label Dec 3, 2024
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.

Thank you @findepi and @Weijun-H -- these are much nicer error messages 💯

This makes for a more actionable error message.
@findepi findepi merged commit 55e56c4 into apache:main Dec 4, 2024
@findepi findepi deleted the findepi/errctx branch December 4, 2024 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

logical-expr Logical plan and expressions optimizer Optimizer rules sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants