Derby
  1. Derby
  2. DERBY-4382

missing documentation from reference guide for ORDER BY null ordering clause (i.e. NULLS FIRST / NULLS LAST )

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 10.4.1.3, 10.4.2.0, 10.5.1.1, 10.5.2.0, 10.5.3.0, 10.6.1.0
    • Fix Version/s: 10.6.1.0
    • Component/s: Documentation
    • Labels:
      None

      Description

      With 10.4.1.3, support was introduced for the SQL standard, 10.10 <sort specification list>:

      <null ordering> ::=
      NULLS FIRST

      NULLS LAST

      See https://issues.apache.org/jira/browse/DERBY-2887.

      However, the documentation doesn't reflect this syntax.
      It should at least get added to the reference guide order by section.

      1. orderBy.diff
        3 kB
        Bryan Pendleton
      2. rrefsqlj13658.html
        7 kB
        Bryan Pendleton

        Issue Links

          Activity

          Myrna van Lunteren created issue -
          Myrna van Lunteren made changes -
          Field Original Value New Value
          Link This issue is related to DERBY-2887 [ DERBY-2887 ]
          Hide
          Myrna van Lunteren added a comment -

          From the duplicate to DERBY-2887, https://issues.apache.org/jira/browse/DERBY-2965:
          "More information about this feature is available at http://wiki.apache.org/db-derby/OLAPNullOrdering"

          Show
          Myrna van Lunteren added a comment - From the duplicate to DERBY-2887 , https://issues.apache.org/jira/browse/DERBY-2965: "More information about this feature is available at http://wiki.apache.org/db-derby/OLAPNullOrdering "
          Hide
          Bryan Pendleton added a comment -

          Hi Myrna, thanks for catching this.

          i will try to draft some docs for us to review.

          Show
          Bryan Pendleton added a comment - Hi Myrna, thanks for catching this. i will try to draft some docs for us to review.
          Bryan Pendleton made changes -
          Assignee Bryan Pendleton [ bryanpendleton ]
          Hide
          Bryan Pendleton added a comment -

          Attached is a proposed documentation patch, and the resulting HTML page for the ORDER BY docs.

          Please let me know what you think.

          Show
          Bryan Pendleton added a comment - Attached is a proposed documentation patch, and the resulting HTML page for the ORDER BY docs. Please let me know what you think.
          Bryan Pendleton made changes -
          Attachment orderBy.diff [ 12420139 ]
          Attachment rrefsqlj13658.html [ 12420140 ]
          Bryan Pendleton made changes -
          Issue & fix info [Patch Available]
          Hide
          Knut Anders Hatlen added a comment -

          Should we also remove the "For example, ..." part in the notes? It states that NULLs are sorted high, which is not necessarily the case anymore.

          Show
          Knut Anders Hatlen added a comment - Should we also remove the "For example, ..." part in the notes? It states that NULLs are sorted high, which is not necessarily the case anymore.
          Hide
          Bryan Pendleton added a comment -

          I agree that comment is a bit confusing. I moved it into the note that discusses the behavior
          when NULLS FIRST/LAST is not specified, but I'm concerned that it may still be confusing.

          We could remove the comment entirely, or we could try to re-word it some more.

          Show
          Bryan Pendleton added a comment - I agree that comment is a bit confusing. I moved it into the note that discusses the behavior when NULLS FIRST/LAST is not specified, but I'm concerned that it may still be confusing. We could remove the comment entirely, or we could try to re-word it some more.
          Hide
          Knut Anders Hatlen added a comment -

          The notes you've added are pretty clear, so I don't think that comment will add much value even if it's reworded. I'd say that we should just remove it.

          Show
          Knut Anders Hatlen added a comment - The notes you've added are pretty clear, so I don't think that comment will add much value even if it's reworded. I'd say that we should just remove it.
          Hide
          Bryan Pendleton added a comment -

          Thanks Knut for the review and suggestions. I removed the confusing sentences
          from the doc page and committed the change to the docs trunk as revision 817266.

          Show
          Bryan Pendleton added a comment - Thanks Knut for the review and suggestions. I removed the confusing sentences from the doc page and committed the change to the docs trunk as revision 817266.
          Bryan Pendleton made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Issue & fix info [Patch Available]
          Fix Version/s 10.6.0.0 [ 12313727 ]
          Resolution Fixed [ 1 ]
          Hide
          Myrna van Lunteren added a comment -

          Confirmed the changes are in the latest alpha docs. Thanks for taking care of it so quickly, Bryan. closing.
          (I guess these changes could be backported to 10.5/10.4...).

          Show
          Myrna van Lunteren added a comment - Confirmed the changes are in the latest alpha docs. Thanks for taking care of it so quickly, Bryan. closing. (I guess these changes could be backported to 10.5/10.4...).
          Myrna van Lunteren made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Gavin made changes -
          Workflow jira [ 12477429 ] Default workflow, editable Closed status [ 12798164 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          1d 21h 38m 1 Bryan Pendleton 21/Sep/09 15:23
          Resolved Resolved Closed Closed
          1d 6h 54m 1 Myrna van Lunteren 22/Sep/09 22:18

            People

            • Assignee:
              Bryan Pendleton
              Reporter:
              Myrna van Lunteren
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development