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

    https://github.com/apache/drill/pull/904#discussion_r149440814
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/fn/impl/TestCastFunctions.java
 ---
    @@ -77,16 +82,23 @@ public void testCastByConstantFolding() throws 
Exception {
     
       @Test // DRILL-3769
       public void testToDateForTimeStamp() throws Exception {
    -    final String query = "select to_date(to_timestamp(-1)) as col \n" +
    -        "from (values(1))";
    +    new MockUp<DateTimeZone>() {
    --- End diff --
    
    This mock is used twice. So let's also move the code into the separate 
method.


---

Reply via email to