From 0227292c85eed8f6acc512bddf49926539a33d67 Mon Sep 17 00:00:00 2001
From: bigben0204 <bigben0204@163.com>
Date: Wed, 27 Sep 2023 16:50:41 +0800
Subject: [PATCH]  [bug](profile) query profile api of fe cann't get result if
 non-root user query on the other fe #24858 (#24914)

Issue Number: #24858

If isAllNode is true, the api should only distribute the query to all fe and do not run checkAuthByUserAndQueryId.
If isAllNode is false, the api queries profile on the fe, at this time the api should run checkAuthByUserAndQueryId.
---
 .../doris/httpv2/rest/manager/QueryProfileAction.java  | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/manager/QueryProfileAction.java b/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/manager/QueryProfileAction.java
index fa597f50c7..e586b99092 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/manager/QueryProfileAction.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/manager/QueryProfileAction.java
@@ -268,10 +268,12 @@ public class QueryProfileAction extends RestBaseController {
             @RequestParam(value = IS_ALL_NODE_PARA, required = false, defaultValue = "true") boolean isAllNode) {
         executeCheckPassword(request, response);
 
-        try {
-            checkAuthByUserAndQueryId(queryId);
-        } catch (AuthenticationException e) {
-            return ResponseEntityBuilder.badRequest(e.getMessage());
+        if (!isAllNode) {
+            try {
+                checkAuthByUserAndQueryId(queryId);
+            } catch (AuthenticationException e) {
+                return ResponseEntityBuilder.badRequest(e.getMessage());
+            }
         }
 
         if (format.equals("text")) {