This is an automated email from the ASF dual-hosted git repository.

gezapeti pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/oozie.git


The following commit(s) were added to refs/heads/master by this push:
     new 34b87e1  OOZIE-3601 Upgrade quartz to 2.3.2 (dengliming via matijhs)
34b87e1 is described below

commit 34b87e1ed8d22315da57acdd108873935c879826
Author: Gezapeti Cseh <gezap...@apache.org>
AuthorDate: Mon Aug 17 21:25:09 2020 +0200

    OOZIE-3601 Upgrade quartz to 2.3.2 (dengliming via matijhs)
---
 core/src/main/java/org/apache/oozie/util/DateUtils.java | 4 ++--
 release-log.txt                                         | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/core/src/main/java/org/apache/oozie/util/DateUtils.java 
b/core/src/main/java/org/apache/oozie/util/DateUtils.java
index e63a1fb..cd51eaa 100644
--- a/core/src/main/java/org/apache/oozie/util/DateUtils.java
+++ b/core/src/main/java/org/apache/oozie/util/DateUtils.java
@@ -143,8 +143,8 @@ public class DateUtils {
         TimeZone tz = TimeZone.getTimeZone(tzId);
 
         // Check whether tzID can handle DST shifts
-        if (!isThreeLetterTZName(tzId)) {
-            LOG.warn("GMT, UTC or Region/City Timezone formats are preferred 
instead of " + tzId);
+        if (isThreeLetterTZName(tzId)) {
+            LOG.warn("GMT, UTC or Region/City Timezone formats are preferred 
instead of [" + tzId + "]");
         }
 
         // If these are not equal, it means that the tzId is not valid 
(invalid tzId's return GMT)
diff --git a/release-log.txt b/release-log.txt
index daca738..a50476d 100644
--- a/release-log.txt
+++ b/release-log.txt
@@ -1,5 +1,6 @@
 -- Oozie 5.3.0 release (trunk - unreleased)
 
+OOZIE-3608 Coordinator emits wrong WARN about TimeZones (larsfrancke via 
gezapeti)
 OOZIE-3601 Upgrade quartz to 2.3.2 (dengliming via matijhs)
 OOZIE-3586 Oozie spark actions using --keytab fail due to duplicate dist. 
cache (jmakai via asalamon74)
 OOZIE-3592 Do not print misleading SecurityException for successful jobs 
(matijhs via asalamon74)

Reply via email to