-
Notifications
You must be signed in to change notification settings - Fork 25.8k
SQL: Remove exception throwing from Analyzer #38208
Copy link
Copy link
Closed
Labels
Description
Currently, there's still a rule that uses exceptions in case of problem - while this achieves the goal, the correct way is to let the Verifier collect all issues and report them in one go.
Additionally the same rule produces an incorrect message:
sql> select gender as g, min(salary) as min, max(salary) as max from test_emp group by 1 order by 2, 3, 2, 4 desc limit 3;
Bad request [line 1:103: Invalid %d specified in OrderBy (valid range is [1, %d])]
Which needs fixing.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Type
Fields
Give feedbackNo fields configured for issues without a type.