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

    https://github.com/apache/incubator-griffin/pull/435#discussion_r224955984
  
    --- 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 --
    
    yes, it is utility function.


---

Reply via email to