Derby
  1. Derby
  2. DERBY-3311

Client ResultSet.getHoldabilty will return incorrect value when the ResultSet is obtained from a procedure call

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 10.4.1.3
    • Fix Version/s: 10.4.1.3
    • Component/s: JDBC, Network Client
    • Labels:
      None

      Description

      Fixing DERBY-3305 (aka DERBY-1111) exposes that the holdability for a result set sent over DRDA is being obtained from the Statement that created the JDBC ResultSet, not its actual holdability. The two can be different when the result set is dynamic result set from a procedure.

      jdbc4.ResultSetTest.getHoldability fails in client mode.

      1. derby_3311_diff.txt
        5 kB
        Daniel John Debrunner

        Issue Links

          Activity

          Hide
          Daniel John Debrunner added a comment -

          Same as DERBY-1101 but DERBY-1101 only fixed the issue for embedded.

          Show
          Daniel John Debrunner added a comment - Same as DERBY-1101 but DERBY-1101 only fixed the issue for embedded.
          Hide
          Daniel John Debrunner added a comment -

          Patch to change the DRDA code to obtain the holdability explicitly from the result set or statement as required as a result set may have a different holdability to its statement.

          Show
          Daniel John Debrunner added a comment - Patch to change the DRDA code to obtain the holdability explicitly from the result set or statement as required as a result set may have a different holdability to its statement.
          Hide
          Daniel John Debrunner added a comment -

          Patch fixes the test, jdbc4._Suite passes as well as jdbcapi.HoldabilityTest and lang.HoldCursorTest.

          Show
          Daniel John Debrunner added a comment - Patch fixes the test, jdbc4._Suite passes as well as jdbcapi.HoldabilityTest and lang.HoldCursorTest.
          Hide
          Dyre Tjeldvoll added a comment -

          All tests pass with this patch for me. +1 to commit.

          Show
          Dyre Tjeldvoll added a comment - All tests pass with this patch for me. +1 to commit.
          Hide
          Dyre Tjeldvoll added a comment -

          Committed revision 613169.

          Show
          Dyre Tjeldvoll added a comment - Committed revision 613169.
          Hide
          Myrna van Lunteren added a comment -

          Can this now be closed?

          Show
          Myrna van Lunteren added a comment - Can this now be closed?
          Hide
          Daniel John Debrunner added a comment -

          Fixed in trunk.

          Show
          Daniel John Debrunner added a comment - Fixed in trunk.

            People

            • Assignee:
              Daniel John Debrunner
              Reporter:
              Daniel John Debrunner
            • Votes:
              1 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development