abstractdog commented on code in PR #5538:
URL: https://github.com/apache/hive/pull/5538#discussion_r1841720276


##########
ql/src/java/org/apache/hadoop/hive/ql/DriverContext.java:
##########
@@ -125,6 +130,12 @@ public QueryPlan getPlan() {
 
   public void setPlan(QueryPlan plan) {
     this.plan = plan;
+    TezTask task = Utilities.getFirstTezTask(plan.getRootTasks()).orElse(null);

Review Comment:
   I see, can you point me to the logs that show this?
   btw I made a slight refactor in this area after pushing the original, 
working version that I used in case of the query history...NPE looks weird as 
we obtain a reference to the RuntimeContext in setPlan...I guess there are 
calls when the passed plan is already null, need to null-check I guess



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to