Derby
  1. Derby
  2. DERBY-4506

Document NATURAL JOIN in the reference manual

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 10.6.1.0
    • Fix Version/s: 10.6.1.0
    • Component/s: Documentation
    • Labels:
      None

      Description

      When support for NATURAL JOIN syntax has been added (DERBY-4495), we should also add a section about it under "JOIN operations" in the reference manual.

      1. rrefsqljnaturaljoin.html
        4 kB
        Knut Anders Hatlen
      2. rrefsqljnaturaljoin.html
        5 kB
        Knut Anders Hatlen
      3. rrefsqlj29840.html
        7 kB
        Knut Anders Hatlen
      4. naturaljoin.diff
        5 kB
        Knut Anders Hatlen
      5. natural_fix1.diff
        0.9 kB
        Knut Anders Hatlen

        Issue Links

          Activity

          Hide
          Knut Anders Hatlen added a comment -

          The attached patch adds syntax description for the NATURAL JOIN operation and links to it from JOIN operations. I've also attached the resulting HTML files.

          Show
          Knut Anders Hatlen added a comment - The attached patch adds syntax description for the NATURAL JOIN operation and links to it from JOIN operations. I've also attached the resulting HTML files.
          Hide
          Rick Hillegas added a comment -

          Thanks for the patch Knut. It is clear and a quick sanity check verifies that it describes the actual behavior of Derby. I didn't check whether this behavior conforms to the SQL Standard. I assume it does and I'm sure that your buddy-tester, Dag, will verify that. +1 to commit these changes. Thanks.

          Show
          Rick Hillegas added a comment - Thanks for the patch Knut. It is clear and a quick sanity check verifies that it describes the actual behavior of Derby. I didn't check whether this behavior conforms to the SQL Standard. I assume it does and I'm sure that your buddy-tester, Dag, will verify that. +1 to commit these changes. Thanks.
          Hide
          Kim Haase added a comment -

          Thanks very much, Knut – the patch looks great.

          Show
          Kim Haase added a comment - Thanks very much, Knut – the patch looks great.
          Hide
          Knut Anders Hatlen added a comment -

          Thanks, Kim and Rick. Committed revision 937168.

          Show
          Knut Anders Hatlen added a comment - Thanks, Kim and Rick. Committed revision 937168.
          Hide
          Knut Anders Hatlen added a comment -

          Reopening the issue because there's an error in the new topic: It says that the first columns in the expansion of an asterisk are the common columns from the first/left table. This is only true for INNER and LEFT OUTER join. For RIGHT OUTER join, we'll take the common columns from the second/right table.

          Show
          Knut Anders Hatlen added a comment - Reopening the issue because there's an error in the new topic: It says that the first columns in the expansion of an asterisk are the common columns from the first/left table. This is only true for INNER and LEFT OUTER join. For RIGHT OUTER join, we'll take the common columns from the second/right table.
          Hide
          Knut Anders Hatlen added a comment -

          Here's a followup patch (natural_fix1.diff) that removes the mentioning of first/left table in the description of how * expands to the common columns. Also added a note about how unqualified references to common columns are resolved.

          Show
          Knut Anders Hatlen added a comment - Here's a followup patch (natural_fix1.diff) that removes the mentioning of first/left table in the description of how * expands to the common columns. Also added a note about how unqualified references to common columns are resolved.
          Hide
          Knut Anders Hatlen added a comment -

          Committed revision 937962.

          Marking as resolved again.

          Show
          Knut Anders Hatlen added a comment - Committed revision 937962. Marking as resolved again.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development