You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2020/09/22 04:05:25 UTC

[GitHub] [incubator-pinot] mayankshriv commented on pull request #6033: Add Broker Reduce Time Log

mayankshriv commented on pull request #6033:
URL: https://github.com/apache/incubator-pinot/pull/6033#issuecomment-696497353


   @mr-agrwal @fx19880617 We do have metrics for each phase of query in the broker side (including scatter/gather, reduce, etc). Adding just the reduce part in the log does not make a lot of sense (because then we could argue why not add other times, and we definitely want to limit log size for high throughput cases).


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org