abstractdog commented on code in PR #5739:
URL: https://github.com/apache/hive/pull/5739#discussion_r2026321165
##########
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java:
##########
@@ -268,8 +268,8 @@ public int execute() {
// Log all the info required to find the various logs for this query
String dagId = this.dagClient.getDagIdentifierString();
String appId = this.dagClient.getSessionIdentifierString();
- LOG.info("HS2 Host: [{}], Query ID: [{}], Dag ID: [{}], DAG App ID:
[{}]", ServerUtils.hostname(), queryId,
- dagId, appId);
+ LOG.info("HS2 Host: [{}], Query ID: [{}], Dag ID: [{}], DAG App ID:
[{}], DAG App address: [{}]",
+ ServerUtils.hostname(), queryId, dagId, appId,
runtimeContext.getAmHostName());
Review Comment:
makes sense, getAmHost works, I changed to it
also removed an unused TezClient import
regarding the confusing sessionRef, agree, but I'm not about to touch that
magic at the moment, created
[HIVE-28877](https://issues.apache.org/jira/browse/HIVE-28877)
--
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: gitbox-unsubscr...@hive.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
