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

    https://github.com/apache/drill/pull/904#discussion_r137777824
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/fn/impl/testing/TestDateConversions.java
 ---
    @@ -99,106 +100,142 @@ public void testPostgresDate() throws Exception {
     
       @Test
       public void testJodaTime() throws Exception {
    -    String query = String.format("SELECT to_time(time1, 'H:m:ss') = "
    -      + "to_time(time2, 'h:m:ssa') as col1, "
    -      + "to_time(time1, 'H:m:ss') = "
    -      + "to_time(time3, 'ssmha') as col2 "
    -      + "from dfs_test.`%s/joda_postgres_time.json`", TEMP_DIR);
    +    Locale defaultLocale = Locale.getDefault();
    +    try {
    +      Locale.setDefault(new Locale("en", "US"));
    --- End diff --
    
    Please replace all occurrences of `new Locale("en", "US")` by `Locale.US` 
in all your changes.


---

Reply via email to