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