Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-1254

Support PreparedStatement.executeLargeBatch

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: avatica-1.8.0
    • Component/s: avatica
    • Labels:
      None

      Description

      In CALCITE-1128 we added support for PreparedStatement.executeBatch. This added ExecuteBatchResult with a field int[] updateCounts.

      I think that field should have been long[] instead. Elsewhere we have been converting update counts from int to long, in line with changes to the JDBC API.

      If changing this field from int[] to long[] will be a breaking change we should consider halting 1.8 to get this in.

        Attachments

        Issue Links

          Activity

            People

            • Assignee:
              elserj Josh Elser
              Reporter:
              julianhyde Julian Hyde

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment