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=16242445#comment-16242445
] 

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_r149440034
  
    --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/fn/interp/TestConstantFolding.java
---
    @@ -117,6 +123,13 @@ public void createFiles(int smallFileLines, int bigFileLines) throws
Exception{
     
       @Test
       public void testConstantFolding_allTypes() throws Exception {
    +    new MockUp<DateTimeUtils>() {
    --- End diff --
    
    I meant to use the same method in all tests where Locale should be mocked. Please replace
it.


> 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