Skip to content

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
25 checks passed
@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