okumin commented on code in PR #4424:
URL: https://github.com/apache/hive/pull/4424#discussion_r1232525651


##########
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/monitoring/RenderStrategy.java:
##########
@@ -161,7 +166,8 @@ public void renderReport(String report) {
       if (hiveServer2InPlaceProgressEnabled) {
         LOGGER.info(report);
       } else {
-        monitor.console.printInfo(report);
+        String time = REPORT_DATE_TIME_FORMATTER.format(LocalDateTime.now());

Review Comment:
   @aturoczy I updated it. I think it is reasonable since it is consistent with 
other places such as [CURRENT_DATE 
UDF](https://github.com/apache/hive/blob/rel/release-4.0.0-alpha-2/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFCurrentDate.java#L55).
   
   I have also verified the property takes effect.
   
   By default, the default one, UTC in this case, is used.
   
   ```
   $ beeline -e 'SELECT sum(price) FROM orders' --hiveconf 
hive.server2.in.place.progress=false
   ...
   INFO  : 2023-06-16 16:56:55,735      Map 1: -/-      Reducer 2: 0/1  
   INFO  : 2023-06-16 16:56:56,240      Map 1: 0/1      Reducer 2: 0/1  
   ```
   
   The same offset is used when we explicitly give `hive.local.time.zone=UTC`.
   
   ```
   $ beeline -e 'SELECT sum(price) FROM orders' --hiveconf 
hive.server2.in.place.progress=false --hiveconf hive.local.time.zone=UTC
   ...
   INFO  : 2023-06-16 16:59:44,985      Map 1: -/-      Reducer 2: 0/1  
   INFO  : 2023-06-16 16:59:45,995      Map 1: 0/1      Reducer 2: 0/1  
   ```
   
   Configuring other time zones, we see a date and time with the offset.
   
   ```
   $ beeline -e 'SELECT sum(price) FROM orders' --hiveconf 
hive.server2.in.place.progress=false --hiveconf 
hive.local.time.zone=America/Los_Angeles
   ...
   INFO  : 2023-06-16 10:02:17,700      Map 1: -/-      Reducer 2: 0/1  
   INFO  : 2023-06-16 10:02:18,712      Map 1: 0/1      Reducer 2: 0/1  
   ...
   $ beeline -e 'SELECT sum(price) FROM orders' --hiveconf 
hive.server2.in.place.progress=false --hiveconf hive.local.time.zone=Asia/Tokyo
   ...
   INFO  : 2023-06-17 02:03:21,041      Map 1: -/-      Reducer 2: 0/1  
   INFO  : 2023-06-17 02:03:22,050      Map 1: 0/1      Reducer 2: 0/1  
   ```
   



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

Reply via email to