ASF GitHub Bot commented on CALCITE-2199:

GitHub user bcogrel opened a pull request:


    [CALCITE-2199] AvaticaResultSet.getStatement() can now throw a SQLException

    See https://issues.apache.org/jira/browse/CALCITE-2199

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/bcogrel/calcite-avatica CALCITE-2199

Alternatively you can review and apply these changes as the patch at:


To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #25
commit ece01794cb26c70bf5ebb072c2f809a262266402
Author: Benjamin Cogrel <benjamin.cogrel@...>
Date:   2018-03-02T16:16:11Z

    [CALCITE-2199] AvaticaResultSet.getStatement() can now throw a SQLException.


> AvaticaResultSet.getStatement() signature is missing SQLException
> -----------------------------------------------------------------
>                 Key: CALCITE-2199
>                 URL: https://issues.apache.org/jira/browse/CALCITE-2199
>             Project: Calcite
>          Issue Type: Improvement
>          Components: avatica
>            Reporter: Benjamin Cogrel
>            Priority: Trivial
> Extensions of the AvaticaResultSet cannot throw a SQLException from the 
> getStatement() method because this checked exception is not part of its 
> signature.
> This forced the Dremio developers to wrapped their SQLException into a 
> runtime exception:
> https://github.com/dremio/dremio-oss/blob/master/client/jdbc/src/main/java/com/dremio/jdbc/impl/DremioResultSetImpl.java#L1153

This message was sent by Atlassian JIRA

Reply via email to