You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kylin.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2018/11/30 10:31:00 UTC

[jira] [Commented] (KYLIN-3187) JDK APIs using the default locale, time zone or character set should be avoided

    [ https://issues.apache.org/jira/browse/KYLIN-3187?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16704544#comment-16704544 ] 

ASF GitHub Bot commented on KYLIN-3187:
---------------------------------------

asfgit commented on issue #368: KYLIN-3187 Forbid calls to JDK APIs that use the default locale
URL: https://github.com/apache/kylin/pull/368#issuecomment-443161048
 
 
   Can one of the admins verify this patch?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


> JDK APIs using the default locale, time zone or character set should be avoided
> -------------------------------------------------------------------------------
>
>                 Key: KYLIN-3187
>                 URL: https://issues.apache.org/jira/browse/KYLIN-3187
>             Project: Kylin
>          Issue Type: Bug
>          Components: REST Service
>            Reporter: Ted Yu
>            Assignee:  Kaige Liu
>            Priority: Major
>              Labels: usability
>             Fix For: v2.6.0
>
>
> Here are a few examples:
> {code}
> server-base/src/main/java/org/apache/kylin/rest/service/JobService.java:        Calendar calendar = Calendar.getInstance();
> storage-hbase/src/main/java/org/apache/kylin/storage/hbase/util/HbaseStreamingInput.java:        Calendar cal = Calendar.getInstance();
> {code}
> Locale should be specified.
> See CALCITE-1667 for related information.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)