real-mj-song commented on PR #14917: URL: https://github.com/apache/pinot/pull/14917#issuecomment-2625386989
@gortiz Oh nice! Your `PinotRuntimeException` and `QException` were exactly what I was looking for 💯 I was not planning to make more changes, but once your PR is merged, I can make a quick PR to replace/remove QueryInfoException.java with your new exception class and slightly change exception catch logic. -- 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
