[GitHub] [incubator-doris] chaoyli commented on a change in pull request #3744: [Feature] Add QueryDetail to store query statistics.

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

chaoyli commented on a change in pull request #3744:
URL: https://github.com/apache/incubator-doris/pull/3744#discussion_r433589126



##########
File path: fe/src/main/java/org/apache/doris/qe/ConnectProcessor.java
##########
@@ -126,6 +129,11 @@ private void auditAfterExec(String origStmt, StatementBase parsedStmt, PQuerySta
                 MetricRepo.HISTO_QUERY_LATENCY.update(elapseMs);
             }
             ctx.getAuditEventBuilder().setIsQuery(true);
+            ctx.getQueryDetail().setEventTime(endTime);

Review comment:
       Because one query will be set when started, and then update when finished.
   If query monitor collect between the two point. The query monitor can use eventTime to distinguish the two event.




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