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

abstractdog 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 ce0823896aa HIVE-21520: Query 'Submit plan' time reported is incorrect 
(#4996) (Butao Zhang reviewed by Laszlo Bodor)
ce0823896aa is described below

commit ce0823896aa6bf3be062764a61ecef1aba309db5
Author: Butao Zhang <zhangbu...@cmss.chinamobile.com>
AuthorDate: Wed Jan 17 17:04:14 2024 +0800

    HIVE-21520: Query 'Submit plan' time reported is incorrect (#4996) (Butao 
Zhang reviewed by Laszlo Bodor)
---
 ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java          | 2 +-
 .../org/apache/hadoop/hive/ql/exec/tez/monitoring/TezJobMonitor.java | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java 
b/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java
index 48e907b20ab..8935843ae83 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java
@@ -258,7 +258,7 @@ public class TezTask extends Task<TezWork> {
         LogUtils.putToMDC(LogUtils.DAGID_KEY, dagId);
 
         // finally monitor will print progress until the job is done
-        TezJobMonitor monitor = new TezJobMonitor(work.getAllWork(), 
dagClient, conf, dag, ctx, counters);
+        TezJobMonitor monitor = new TezJobMonitor(work.getAllWork(), 
dagClient, conf, dag, ctx, counters, perfLogger);
         rc = monitor.monitorExecution();
 
         if (rc != 0) {
diff --git 
a/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/monitoring/TezJobMonitor.java 
b/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/monitoring/TezJobMonitor.java
index 17ab6e39397..fbbbf189e48 100644
--- 
a/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/monitoring/TezJobMonitor.java
+++ 
b/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/monitoring/TezJobMonitor.java
@@ -77,7 +77,7 @@ public class TezJobMonitor {
   private static final int MAX_RETRY_INTERVAL = 2500;
   private static final int MAX_RETRY_FAILURES = (MAX_RETRY_INTERVAL / 
MAX_CHECK_INTERVAL) + 1;
 
-  private final PerfLogger perfLogger = SessionState.getPerfLogger();
+  private final PerfLogger perfLogger;
   private static final List<DAGClient> shutdownList;
   private final List<BaseWork> topSortedWorks;
 
@@ -117,7 +117,7 @@ public class TezJobMonitor {
   private final TezCounters counters;
 
   public TezJobMonitor(List<BaseWork> topSortedWorks, final DAGClient 
dagClient, HiveConf conf, DAG dag,
-    Context ctx, final TezCounters counters) {
+    Context ctx, final TezCounters counters, PerfLogger perfLogger) {
     this.topSortedWorks = topSortedWorks;
     this.dagClient = dagClient;
     this.hiveConf = conf;
@@ -126,6 +126,7 @@ public class TezJobMonitor {
     console = SessionState.getConsole();
     updateFunction = updateFunction();
     this.counters = counters;
+    this.perfLogger = perfLogger;
   }
 
   private RenderStrategy.UpdateFunction updateFunction() {

Reply via email to