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. rrefsqlj13658.html
        7 kB
        Bryan Pendleton
      2. orderBy.diff
        3 kB
        Bryan Pendleton

        Issue Links

          Activity

          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.
          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.
          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.
          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...).

            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