albertobastos opened a new pull request, #15532: URL: https://github.com/apache/pinot/pull/15532
This PR aims to reduce how we spam the logs under certain query error scenarios, usually including big stack traces that are either irrelevant or redundant. Main changes include: - Consider `EXECUTION_TIMEOUT` a _Green error_ instead of a _Yellow_ one. That way the broker will not log its own stack trace, which is irrelevant when the timeout took place in one or several servers. - Stage cancellation catches are now re-thrown as `QueryCancelledException` and server will decide not to print the stack trace again when that's the type of exception catched. That way in a timeout, the only stage that will actually log the stack trace is the one that really timed out and parent stages will stop printing redundant info. -- 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
