architjainjain commented on code in PR #6501:
URL: https://github.com/apache/hive/pull/6501#discussion_r3713606881


##########
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/monitoring/TezJobMonitor.java:
##########
@@ -144,6 +153,99 @@ private RenderStrategy.UpdateFunction updateFunction() {
         : new RenderStrategy.LogToFileFunction(this, perfLogger);
   }
 
+  /**
+   * Initializes the YARN queue metrics collector based on configuration.
+   *
+   * <p>This method implements the Null Object pattern - it always returns a 
non-null collector.
+   * When metrics collection is disabled or initialization fails, a no-op 
collector is returned.

Review Comment:
   done



-- 
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