kfaraz opened a new pull request, #16186: URL: https://github.com/apache/druid/pull/16186
Looking at other failure types such as `InvalidInput`, `InvalidSqlInput`, etc. it makes to have the error code be fixed for a certain of failure. The only two usages of the method `InternalServerError.exception()` were actually passing a whole long message in the error code. This would result in the `errorCode` field being populated with the error message and the `errorMsg` field itself being null. ### Changes - Use error code `internalServerError` for failures of this type - Remove the error code argument from `InternalServerError.exception()` methods thus fixing the above mentioned bug in the callers. -- 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]
