[GitHub] incubator-griffin pull request #435: [GRIFFIN-206] fix job timezone when cre...

[ahutsunshine]

Github user ahutsunshine commented on a diff in the pull request:

    https://github.com/apache/incubator-griffin/pull/435#discussion_r224951998
  
    --- Diff: service/src/main/java/org/apache/griffin/core/util/TimeUtil.java 
---
    @@ -148,4 +149,11 @@ private static String refreshEscapeHashTag(String str) 
{
             return str.replaceAll(escapeHashTagPattern, hashTag);
         }
     
    +    public static TimeZone getTimeZone(String timezone) {
    +        if (StringUtils.isEmpty(timezone)) {
    +            return TimeZone.getDefault();
    +        }
    +        return TimeZone.getTimeZone(timezone);
    +    }
    +
     }
    --- End diff --
    
    @chemikadze 
    I suggest this code be kept in its original place. Currently just modify ui 
code to fix bug.


---