[GitHub] [pinot] walterddr commented on issue #10921: Improve Syntax Error Messages

["walterddr (via GitHub)" <gi...@apache.org>]

walterddr commented on issue #10921:
URL: https://github.com/apache/pinot/issues/10921#issuecomment-1607754155

   I think the error messages you shared are generated internally. but it is not piped back to the end-user. 
   it has to do with 
   1. the boxing of the controller handler error on top of the calcite error; and 
   2. the `QueryException.getException` truncates the error messages to only 5 lines (per caused-by i think)
   3. (potentially) the UI renders only the first 5 lines
   thus for some reason the "Caused by part of the error is not returned"
   
   so i think the next step is:
   1. verify with browser plugin and see what the actual json payload is returned to the browser (to see if it is the UI render problem)
   2. verify the error thrown in controller to see if it actually encodes the error message to the per-caused-by level)


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org