Derby
  1. Derby
  2. DERBY-4682

Inaccurate description of how a statement is completed in Developer's guide

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 10.6.1.0
    • Fix Version/s: 10.6.2.1, 10.7.1.1
    • Component/s: Documentation
    • Labels:
      None

      Description

      The Developer's guide says this about how a statement is completed and
      auto-committed:
      http://db.apache.org/derby/docs/10.6/devguide/cdevconcepts29416.html

      > In the case of a statement returning a ResultSet, the statement
      > completes when the last row of the ResultSet has been retrieved or
      > the ResultSet has been closed explicitly.

      If a ResultSet is scrollable, the statement is not complete when the
      last row has been retrieved. See the discussion here:

      http://mail-archives.apache.org/mod_mbox/db-derby-dev/201005.mbox/%3C4C03C7AD.8070608@Sun.COM%3E

      1. DERBY-4682.diff
        1 kB
        Kim Haase
      2. cdevconcepts29416.html
        11 kB
        Kim Haase

        Activity

        Hide
        Kim Haase added a comment -

        Would it be reasonable to change the text to say,

        In the case of a statement returning a forward only ResultSet, the statement completes when the last row of the ResultSet has been retrieved or the ResultSet has been closed explicitly. In the case of a statement returning a scrollable ResultSet, the statement completes only when the ResultSet has been closed explicitly.

        I'm working on a patch with this wording – please let me know if any changes are needed.

        Show
        Kim Haase added a comment - Would it be reasonable to change the text to say, In the case of a statement returning a forward only ResultSet, the statement completes when the last row of the ResultSet has been retrieved or the ResultSet has been closed explicitly. In the case of a statement returning a scrollable ResultSet, the statement completes only when the ResultSet has been closed explicitly. I'm working on a patch with this wording – please let me know if any changes are needed.
        Hide
        Kim Haase added a comment -

        Attaching DERBY-4682.diff and cdevconcepts29416.html, with the change to the "Using auto-commit" topic. Please let me know if changes are needed.

        Show
        Kim Haase added a comment - Attaching DERBY-4682 .diff and cdevconcepts29416.html, with the change to the "Using auto-commit" topic. Please let me know if changes are needed.
        Hide
        Knut Anders Hatlen added a comment -

        Thanks, Kim! The new wording looks fine to me. +1 to commit.

        Show
        Knut Anders Hatlen added a comment - Thanks, Kim! The new wording looks fine to me. +1 to commit.
        Hide
        Kim Haase added a comment -

        Thanks, Knut!

        Committed patch DERBY-4682.diff to documentation trunk at revision 950199.
        Merged to 10.6 doc branch at revision 950200.

        Show
        Kim Haase added a comment - Thanks, Knut! Committed patch DERBY-4682 .diff to documentation trunk at revision 950199. Merged to 10.6 doc branch at revision 950200.
        Hide
        Kim Haase added a comment -

        Fix has appeared in latest alpha docs.

        Show
        Kim Haase added a comment - Fix has appeared in latest alpha docs.

          People

          • Assignee:
            Kim Haase
            Reporter:
            Knut Anders Hatlen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development