belugabehr commented on a change in pull request #1963:
URL: https://github.com/apache/hive/pull/1963#discussion_r573980532
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java
##########
@@ -137,6 +139,8 @@ public int execute() {
Context ctx = null;
Ref<TezSessionState> sessionRef = Ref.from(null);
+ final String queryId = HiveConf.getVar(conf,
HiveConf.ConfVars.HIVEQUERYID);
Review comment:
In working with this code, I think the Hive query ID is passed around in
a few different ways. There can be follow up work to unify/simplify this, but
I would not want to change it too much in this unrelated ticket.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]