xy720 commented on a change in pull request #7030:
URL: https://github.com/apache/incubator-doris/pull/7030#discussion_r747347048



##########
File path: fe/fe-core/src/main/java/org/apache/doris/qe/ConnectProcessor.java
##########
@@ -123,6 +124,15 @@ private void auditAfterExec(String origStmt, StatementBase 
parsedStmt, Data.PQue
             .setStmtId(ctx.getStmtId())
             .setQueryId(ctx.queryId() == null ? "NaN" : 
DebugUtil.printId(ctx.queryId()));
 
+        if (statistics != null && statistics.getNodesStatisticsCount() != 0) {

Review comment:
       Now the calculation will be completed in result_sink in the top level 
fragment at the be side, but the collected node statistics will still be 
transmitted to Fe.

##########
File path: fe/fe-core/src/main/java/org/apache/doris/qe/ConnectProcessor.java
##########
@@ -123,6 +124,15 @@ private void auditAfterExec(String origStmt, StatementBase 
parsedStmt, Data.PQue
             .setStmtId(ctx.getStmtId())
             .setQueryId(ctx.queryId() == null ? "NaN" : 
DebugUtil.printId(ctx.queryId()));
 
+        if (statistics != null && statistics.getNodesStatisticsCount() != 0) {

Review comment:
       Now the calculation will be completed in result sink in the top level 
fragment at the be side, but the collected node statistics will still be 
transmitted to Fe.




-- 
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...@doris.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to