[GitHub] [incubator-pinot] mqliang commented on a change in pull request #6711: Set thread cpu time in setExecutionStatistics()

[GitBox <gi...@apache.org>]

mqliang commented on a change in pull request #6711:
URL: https://github.com/apache/incubator-pinot/pull/6711#discussion_r600838397



##########
File path: pinot-core/src/main/java/org/apache/pinot/core/operator/combine/SelectionOnlyCombineOperator.java
##########
@@ -60,7 +60,7 @@ protected IntermediateResultsBlock getNextBlock() {
     // For LIMIT 0 query, only process one segment to get the data schema
     if (_numRowsToKeep == 0) {
       IntermediateResultsBlock resultsBlock = (IntermediateResultsBlock) _operators.get(0).nextBlock();
-      CombineOperatorUtils.setExecutionStatistics(resultsBlock, _operators);
+      CombineOperatorUtils.setExecutionStatistics(resultsBlock, _operators, 0);

Review comment:
       Yes, we don't need to time the execution time here since this is a single thread case. For all single thread execution time, timing is conducted at `InstanceResponseBlock.getNetxBlock()`




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