From ab8adea39ff66c9c3a3ccf8c974c4e6dc60102d4 Mon Sep 17 00:00:00 2001 From: Mingyu Chen Date: Thu, 27 Apr 2023 11:05:00 +0800 Subject: [PATCH] [fix](load-profile) fix show load profile return emoty result (#19125) Fix 2 bugs of show load profile: For broker load, the second level should be the sub task' id. show load profile stmt should be forwarded to master FE to execute. --- .../java/org/apache/doris/analysis/ShowLoadProfileStmt.java | 5 +++++ .../src/main/java/org/apache/doris/qe/ShowExecutor.java | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/analysis/ShowLoadProfileStmt.java b/fe/fe-core/src/main/java/org/apache/doris/analysis/ShowLoadProfileStmt.java index 91cbfc3c4e..88b960ce97 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/analysis/ShowLoadProfileStmt.java +++ b/fe/fe-core/src/main/java/org/apache/doris/analysis/ShowLoadProfileStmt.java @@ -150,5 +150,10 @@ public class ShowLoadProfileStmt extends ShowStmt { return null; } } + + @Override + public RedirectStatus getRedirectStatus() { + return RedirectStatus.FORWARD_NO_SYNC; + } } diff --git a/fe/fe-core/src/main/java/org/apache/doris/qe/ShowExecutor.java b/fe/fe-core/src/main/java/org/apache/doris/qe/ShowExecutor.java index 4c9757616e..359092f16c 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/qe/ShowExecutor.java +++ b/fe/fe-core/src/main/java/org/apache/doris/qe/ShowExecutor.java @@ -2160,7 +2160,7 @@ public class ShowExecutor { } case FRAGMENTS: { ProfileTreeNode treeRoot = ProfileManager.getInstance().getFragmentProfileTree(showStmt.getJobId(), - showStmt.getJobId()); + showStmt.getTaskId()); if (treeRoot == null) { throw new AnalysisException("Failed to get fragment tree for load: " + showStmt.getJobId()); }