stalary commented on code in PR #11386:
URL: https://github.com/apache/doris/pull/11386#discussion_r935032627


##########
fe/fe-core/src/main/java/org/apache/doris/qe/ConnectProcessor.java:
##########
@@ -420,6 +420,12 @@ private void finalizeCommand() throws IOException {
 
         MysqlChannel channel = ctx.getMysqlChannel();
         channel.sendAndFlush(packet);
+        // note(wb) we should write profile after return result to mysql client
+        // because write profile maybe take too much time
+        // explain query stmt do not have profile
+        if (executor != null && !executor.getParsedStmt().isExplain()) {
+            executor.writeProfile(true);

Review Comment:
   > I want to write profile after query result return to client.
   I know the idea, but I mean the side effect of this move is to expand the 
scope of the profile. Such as insert, use etc. also write profile.



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