kgyrtkirk commented on a change in pull request #1870:
URL: https://github.com/apache/hive/pull/1870#discussion_r563673300
##########
File path:
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/monitoring/RenderStrategy.java
##########
@@ -39,7 +40,16 @@
}
private abstract static class BaseUpdateFunction implements UpdateFunction {
- private static final int PRINT_INTERVAL = 3000;
+ private static final long PRINT_INTERVAL;
+
+ static {
Review comment:
This approach seems to be great!
Yes you could get it from SS as well ; but in some cases there might be no
seesion in some cases - which should be handled differently.
Thank you!
----------------------------------------------------------------
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
