You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2020/06/25 19:17:08 UTC

[GitHub] [incubator-doris] lxxstc opened a new issue #3950: Please support timezone as more as possible

lxxstc opened a new issue #3950:
URL: https://github.com/apache/incubator-doris/issues/3950


   The time zone of my server is UTC. so, It will throw an exception like "Unknown or incorrect time zone: 'UTC'", when I perform some DDL. Please support the timezone as more as possible, at least including UTC.
   Below is git diff between my workspace and branch master
   
   index 21612bfc..6dcfe0ce 100644
   --- a/fe/src/main/java/org/apache/doris/common/util/TimeUtils.java
   +++ b/fe/src/main/java/org/apache/doris/common/util/TimeUtils.java
   @@ -256,7 +256,7 @@ public class TimeUtils {
                Matcher matcher = TIMEZONE_OFFSET_FORMAT_REG.matcher(value);
                // it supports offset and region timezone type, "CST" use here is compatibility purposes.
                boolean match = matcher.matches();
   -            if (!value.contains("/") && !value.equals("CST") && !match) {
   +            if (!value.contains("/") && !value.equals("UTC") && !value.equals("CST") && !match) {
                    ErrorReport.reportDdlException(ErrorCode.ERR_UNKNOWN_TIME_ZONE, value);
                }
                if (match) {
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [incubator-doris] lxxstc commented on issue #3950: Please support timezone as more as possible

Posted by GitBox <gi...@apache.org>.
lxxstc commented on issue #3950:
URL: https://github.com/apache/incubator-doris/issues/3950#issuecomment-649775044


   I have checked the output of  `timedatectl list-timezones`, only `UTC` has been lost.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [incubator-doris] lxxstc commented on issue #3950: Please support timezone as more as possible

Posted by GitBox <gi...@apache.org>.
lxxstc commented on issue #3950:
URL: https://github.com/apache/incubator-doris/issues/3950#issuecomment-650689189


   another way to fix this issue with the command below
   ```
   # timedatectl set-timezone Atlantic/Reykjavik
   ```
   replace the time zone UTC with time zone Atlantic/Reykjavik


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [incubator-doris] morningman commented on issue #3950: Please support timezone as more as possible

Posted by GitBox <gi...@apache.org>.
morningman commented on issue #3950:
URL: https://github.com/apache/incubator-doris/issues/3950#issuecomment-650701709


   Could you submit a PR to add UTC for us?


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org