gianm opened a new pull request, #19234: URL: https://github.com/apache/druid/pull/19234
This patch makes various changes to MSQ exception handling, with the goal of better compatibility with DruidExceptions: 1) Add `DruidExceptionFault`, which allows including `DruidException` in `MSQErrorReport` without losing any information. 2) Make `MSQFault#toDruidException` abstract so all fault implementations must reference a specific category and persona. Changed the category and persona of various faults, in situations where the inherited default was not ideal. 3) `QueryRuntimeFault` no longer generated, instead we're relying fully on `DruidException` for exceptions that originate in the query runtime. 4) Retain original `DruidException`, rather than wrapping and re-throwing, in `ControllerImpl`, `SegmentGeneratorFrameProcessor`, and `RowBasedFrameWriter`. 5) Update `Either` to throw non-DEVELOPER `DruidException` as-is, rather than wrapping them. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
