gortiz commented on PR #14395: URL: https://github.com/apache/pinot/pull/14395#issuecomment-2458921898
This may be a good temporal solution, but in general printing messages as they are is a bad practice. It can leak tons of information that shouldn't be leaked. The ideal solution would be to have our own exception (like PinotException) with a `getUserMessage()` method that can be trusted. Given we are very far away from that, the solution proposed in this PR seems good enough. -- 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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
