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

[FEATURE] Include expression in message when expression evaluation throws an exception #3131

Open
normanj-bitquill opened this issue Oct 28, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@normanj-bitquill
Copy link
Contributor

Is your feature request related to a problem?
When evaluating an expression and it throws an exception, the message does not include the expression that failed. A query may contain many expressions, so it can be unclear to the user which expression they need to fix.

What solution would you like?
Include the failing expression or sub expression in the message of the exception that is thrown.

What alternatives have you considered?
Point to the position in the input query where the failing expression is found.

Do you have any additional context?
Example query and error:

SELECT x, ABS(y) FROM test2;
{
  "error": {
    "reason": "Invalid SQL query",
    "details": "invalid to get longValue from value of type ARRAY",
    "type": "ExpressionEvaluationException"
  },
  "status": 400
}

As you can see, it is unclear that the problem is from ABS(y).

@normanj-bitquill normanj-bitquill added enhancement New feature or request untriaged labels Oct 28, 2024
@andrross
Copy link
Member

[Catch All Triage - 1, 2, 3, 4, 5]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants