Uploaded image for project: 'Derby'
  1. Derby
  2. DERBY-4506

Document NATURAL JOIN in the reference manual

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: 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. natural_fix1.diff
        0.9 kB
        Knut Anders Hatlen
      2. naturaljoin.diff
        5 kB
        Knut Anders Hatlen
      3. rrefsqlj29840.html
        7 kB
        Knut Anders Hatlen
      4. rrefsqljnaturaljoin.html
        5 kB
        Knut Anders Hatlen
      5. rrefsqljnaturaljoin.html
        4 kB
        Knut Anders Hatlen

        Issue Links

          Activity

          Hide
          knutanders 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
          knutanders 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
          rhillegas 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
          rhillegas 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
          chaase3 Kim Haase added a comment -

          Thanks very much, Knut – the patch looks great.

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

          Thanks, Kim and Rick. Committed revision 937168.

          Show
          knutanders Knut Anders Hatlen added a comment - Thanks, Kim and Rick. Committed revision 937168.
          Hide
          knutanders 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
          knutanders 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
          knutanders 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
          knutanders 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
          knutanders Knut Anders Hatlen added a comment -

          Committed revision 937962.

          Marking as resolved again.

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

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development