[
https://issues.apache.org/jira/browse/HIVE-26290?focusedWorklogId=779396&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-779396
]
ASF GitHub Bot logged work on HIVE-26290:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 08/Jun/22 10:07
Start Date: 08/Jun/22 10:07
Worklog Time Spent: 10m
Work Description: zabetak commented on code in PR #3342:
URL: https://github.com/apache/hive/pull/3342#discussion_r892173361
##########
ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFFromUnixTime.java:
##########
@@ -99,7 +98,6 @@ public void configure(MapredContext context) {
if (context != null) {
String timeZoneStr = HiveConf.getVar(context.getJobConf(),
HiveConf.ConfVars.HIVE_LOCAL_TIME_ZONE);
timeZone = TimestampTZUtil.parseTimeZone(timeZoneStr);
- FORMATTER.withZone(timeZone);
Review Comment:
Hey @ayushtkn did you have a chance to have a second look on this PR?
Issue Time Tracking
-------------------
Worklog Id: (was: 779396)
Time Spent: 40m (was: 0.5h)
> Remove useless calls to DateTimeFormatter#withZone without assignment
> ---------------------------------------------------------------------
>
> Key: HIVE-26290
> URL: https://issues.apache.org/jira/browse/HIVE-26290
> Project: Hive
> Issue Type: Task
> Components: HiveServer2
> Reporter: Stamatis Zampetakis
> Assignee: Stamatis Zampetakis
> Priority: Major
> Labels: pull-request-available
> Time Spent: 40m
> Remaining Estimate: 0h
>
> There are some places in the code calling \{{DateTimeFormatter#withZone}}
> without assigning the result anywhere. This basically makes the call useless
> since the method does not modify the formatter instance but always creates a
> new one.
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
