[jira] [Commented] (CALCITE-2140) Basic implementation of Statement#getMoreResults()

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CALCITE-2140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16382824#comment-16382824 ] 

ASF GitHub Bot commented on CALCITE-2140:
-----------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/calcite-avatica/pull/24


> Basic implementation of Statement#getMoreResults()
> --------------------------------------------------
>
>                 Key: CALCITE-2140
>                 URL: https://issues.apache.org/jira/browse/CALCITE-2140
>             Project: Calcite
>          Issue Type: Improvement
>          Components: avatica
>            Reporter: Laurent Goujon
>            Priority: Minor
>
> Methods {{Statement#getMoreResults}} and {{Statement#getMoreResults(int)}} are currently throwing a {{SQLFeatureNotSupportedException}}, but at least one tool (DataGrip) is calling the first method after each SQL request.
> Avatica doesn't support multiple {{ResultSet}} for a given {{Statement}} (see [https://github.com/apache/calcite-avatica/blob/master/core/src/main/java/org/apache/calcite/avatica/AvaticaDatabaseMetaData.java#L1148]), so a basic implementation could close the current open result set (if any), and return {{false}} for both methods (unless the int argument is equal to {{Statement.KEEP_CURRENT_RESULT}} or {{Statement.CLOSE_ALL_RESULTS}}).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)