(hive) branch master updated: HIVE-27796: Allow printing non-extended explain for non-WebUI output (Ryu Kobayashi, reviewed by Denys. Kuzmenko)

[dkuzmenko]

This is an automated email from the ASF dual-hosted git repository.

dkuzmenko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new d283e20860a HIVE-27796: Allow printing non-extended explain for 
non-WebUI output (Ryu Kobayashi, reviewed by Denys. Kuzmenko)
d283e20860a is described below

commit d283e20860a59de38c9bc4a0487ae1efcb802a4d
Author: Ryu Kobayashi <beter....@gmail.com>
AuthorDate: Tue Mar 25 17:54:33 2025 +0900

    HIVE-27796: Allow printing non-extended explain for non-WebUI output (Ryu 
Kobayashi, reviewed by Denys. Kuzmenko)
    
    Closes #4799
---
 ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java 
b/ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java
index 5baa5c09920..4d7b5775c7f 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java
@@ -1434,10 +1434,10 @@ public static String 
getExplainOutput(BaseSemanticAnalyzer sem, QueryPlan plan,
     PrintStream ps = new PrintStream(baos);
     try {
       List<Task<?>> rootTasks = sem.getAllRootTasks();
+      boolean isExtended = 
conf.getBoolVar(ConfVars.HIVE_LOG_EXPLAIN_OUTPUT_INCLUDE_EXTENDED);
       if (conf.getBoolVar(ConfVars.HIVE_SERVER2_WEBUI_SHOW_GRAPH)) {
         JSONObject jsonPlan = task.getJSONPlan(
-            null, rootTasks, sem.getFetchTask(), true,
-            
conf.getBoolVar(ConfVars.HIVE_LOG_EXPLAIN_OUTPUT_INCLUDE_EXTENDED), true, 
sem.getCboInfo(),
+            null, rootTasks, sem.getFetchTask(), true, isExtended, true, 
sem.getCboInfo(),
             plan.getOptimizedCBOPlan(), plan.getOptimizedQueryString());
         if (jsonPlan.getJSONObject(ExplainTask.STAGE_DEPENDENCIES) != null &&
             jsonPlan.getJSONObject(ExplainTask.STAGE_DEPENDENCIES).length() <=
@@ -1447,7 +1447,7 @@ public static String 
getExplainOutput(BaseSemanticAnalyzer sem, QueryPlan plan,
           ret = null;
         }
       } else {
-        task.getJSONPlan(ps, rootTasks, sem.getFetchTask(), false, true, true, 
sem.getCboInfo(),
+        task.getJSONPlan(ps, rootTasks, sem.getFetchTask(), false, isExtended, 
true, sem.getCboInfo(),
             plan.getOptimizedCBOPlan(), plan.getOptimizedQueryString());
         ret = baos.toString();
       }