You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/09/10 02:33:41 UTC

[GitHub] [druid] gianm opened a new pull request #10377: Include Sequence-building time in CPU time metric.

gianm opened a new pull request #10377:
URL: https://github.com/apache/druid/pull/10377


   Meaningful work can be done while building Sequences, and we should
   count this work. On the Broker, this includes subquery processing
   work done by the mergeResults call of the GroupByQueryQueryToolChest.


----------------------------------------------------------------
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@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] gianm commented on pull request #10377: Include Sequence-building time in CPU time metric.

Posted by GitBox <gi...@apache.org>.
gianm commented on pull request #10377:
URL: https://github.com/apache/druid/pull/10377#issuecomment-692168297


   > Nice. It would be useful to call this out in the Release notes as well.
   
   Good point, I added the "Release Notes" label.
   
   I haven't had time to fix the coverage issue yet, but I'll be able to get to it soonish.


----------------------------------------------------------------
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@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] gianm commented on pull request #10377: Include Sequence-building time in CPU time metric.

Posted by GitBox <gi...@apache.org>.
gianm commented on pull request #10377:
URL: https://github.com/apache/druid/pull/10377#issuecomment-689960942


   ![image](https://user-images.githubusercontent.com/1214075/92679657-e8769080-f2dd-11ea-88f4-f10b4295c1a7.png)
   


----------------------------------------------------------------
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@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] asdf2014 merged pull request #10377: Include Sequence-building time in CPU time metric.

Posted by GitBox <gi...@apache.org>.
asdf2014 merged pull request #10377:
URL: https://github.com/apache/druid/pull/10377


   


----------------------------------------------------------------
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@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] a2l007 commented on pull request #10377: Include Sequence-building time in CPU time metric.

Posted by GitBox <gi...@apache.org>.
a2l007 commented on pull request #10377:
URL: https://github.com/apache/druid/pull/10377#issuecomment-692096884


   Nice. It would be useful to call this out in the Release notes as well.


----------------------------------------------------------------
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@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org