[GitHub] drill pull request #1198: DRILL-6294: Changes to support Calcite 1.16.0

[vvysotskyi]

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

    https://github.com/apache/drill/pull/1198#discussion_r178404654
  
    --- Diff: 
exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillResultSetImpl.java ---
    @@ -1142,16 +1142,8 @@ public void moveToCurrentRow() throws SQLException {
       }
     
       @Override
    -  public AvaticaStatement getStatement() {
    -    try {
    -      throwIfClosed();
    -    } catch (AlreadyClosedSqlException e) {
    -      // Can't throw any SQLException because AvaticaConnection's
    -      // getStatement() is missing "throws SQLException".
    -      throw new RuntimeException(e.getMessage(), e);
    -    } catch (SQLException e) {
    -      throw new RuntimeException(e.getMessage(), e);
    -    }
    +  public AvaticaStatement getStatement() throws SQLException {
    +    throwIfClosed();
    --- End diff --
    
    Thanks, it looks clearer now.


---