drill-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DRILL-5717) change some date time unit cases with specific timezone or Local
Date Tue, 07 Nov 2017 17:22:00 GMT

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

ASF GitHub Bot commented on DRILL-5717:
---------------------------------------

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

    https://github.com/apache/drill/pull/904#discussion_r149440392
  
    --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/fn/impl/testing/TestDateConversions.java
---
    @@ -225,4 +243,16 @@ public void testPostgresDateFormatError() throws Exception {
           throw e;
         }
       }
    +
    +  /**
    +   * mock current locale to US
    +   */
    +  private void mockUSLocale() {
    --- End diff --
    
    Please move this method into ExecTest class and make it public and static.


> change some date time unit cases with specific timezone or Local
> ----------------------------------------------------------------
>
>                 Key: DRILL-5717
>                 URL: https://issues.apache.org/jira/browse/DRILL-5717
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Tools, Build & Test
>    Affects Versions: 1.9.0, 1.11.0
>            Reporter: weijie.tong
>
> Some date time test cases like  JodaDateValidatorTest  is not Local independent .This
will cause other Local's users's test phase to fail. We should let these test cases to be
Local env independent.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message