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

    https://github.com/apache/drill/pull/940#discussion_r138497894
  
    --- Diff: exec/jdbc/src/test/java/org/apache/drill/jdbc/ConnectionTest.java 
---
    @@ -57,7 +61,9 @@ public static void setUpConnection() throws SQLException {
         // Connection--and other JDBC objects--on test method failure, but 
this test
         // class uses some objects across methods.)
         Driver.load();
    -    connection = DriverManager.getConnection( "jdbc:drill:zk=local" );
    +    Properties properties = new Properties();
    +    properties.setProperty(OptionValidator.OPTION_DEFAULTS_ROOT + 
ExecConstants.CREATE_PREPARE_STATEMENT_TIMEOUT_MILLIS, "30000");
    --- End diff --
    
    I made ExecConstants a file class with a private constructor and added the 
static method there.


---

Reply via email to