You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by yi...@apache.org on 2023/04/27 03:05:07 UTC
[doris] branch master updated: [fix](load-profile) fix show load profile return emoty result (#19125)
This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new ab8adea39f [fix](load-profile) fix show load profile return emoty result (#19125)
ab8adea39f is described below
commit ab8adea39ff66c9c3a3ccf8c974c4e6dc60102d4
Author: Mingyu Chen <mo...@163.com>
AuthorDate: Thu Apr 27 11:05:00 2023 +0800
[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.
---
.../src/main/java/org/apache/doris/analysis/ShowLoadProfileStmt.java | 5 +++++
fe/fe-core/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());
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org