imply-cheddar commented on PR #14975: URL: https://github.com/apache/druid/pull/14975#issuecomment-1719226826
The `QueryCompat` stuff is a compatibility layer with the way that the old `QueryException` worked and an attempt to map it onto `DruidException`. `QueryException`s are/were exceptions that are serialized across the wire, deserialized in the brokers and then converted back into exceptions. IIRC, they don't include the stack traces because the historical-side stacktrace doesn't make sense on the broker. The correct approach to fix odd exception reporting here is to go to the source of the exception and turn it into a meaningful `DruidException` which will cause the higher stacks to use the `DruidException` handling code instead. -- 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]
