mcvsubbu commented on pull request #8433: URL: https://github.com/apache/pinot/pull/8433#issuecomment-1083397010
> A while ago we had aa problem that was a performance regression for some use case. It turned out that we could have caught it if we had checked on number of documents scanned. This was way before compat tests were built (you were there as well :) ). So, in compat tests, it was decided to check for regression. I still think it is not bad to check for num docs scanned (and perhaps other stats as well) in this manner (where we allow them to get better). -- 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
