pgaref commented on a change in pull request #1963:
URL: https://github.com/apache/hive/pull/1963#discussion_r573937582
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java
##########
@@ -236,6 +239,10 @@ public int execute() {
throw new HiveException("Operation cancelled");
}
+ // Log all the info required to find the various logs for this query
+ LOG.info("HS2 Host: [{}], Query ID: [{}], Dag ID: [{}], DAG Session
ID: [{}]", getHostNameIP(), queryId,
Review comment:
I would change that to something like "Executing query with Id {} from
dag {} by session {} "
Not sure I would add the hostName IP here as this can be quite confusing,
especially when running mutiple containers on the same node (like kubernetes).
Maybe we already have better info for that on a Task level ?
----------------------------------------------------------------
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]
