lriggs opened a new issue, #50111:
URL: https://github.com/apache/arrow/issues/50111

   ### Describe the enhancement requested
   
   ## Motivation
   
   Gandiva's runtime errors are surfaced to SQL users via 
`ExecutionContext::set_error_msg`. An audit of all ~90 call sites in 
`cpp/src/gandiva/` turned up two recurring problems:
   
   1. **No SQL function name in the message.** Users see `"divide by zero 
error"` or `"Output buffer length can't be negative"` with no indication of 
which SQL function produced it, making errors hard to localize in long queries.
   2. **The offending value is not echoed.** Many messages reject a value 
(invalid weekday, bad boolean string, out-of-range index, …) without telling 
the user what was supplied.
   
   A good runtime error should answer four questions: *which function*, *what 
went wrong*, *what value triggered it*, *what's the valid range*. This PR moves 
the highest-impact messages toward that bar.
   
   
   ### Component(s)
   
   Gandiva


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to