richardstartin commented on pull request #8137: URL: https://github.com/apache/pinot/pull/8137#issuecomment-1031973175
> Agree on avoiding the exception in normal execution path. Do we have some performance number back this change? I had numbers for a few of these PRs (which have been submitted separately because some users prefer tightly scoped changes) and this was just an incremental improvement. I can measure this separately. -- 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