Derby
  1. Derby
  2. DERBY-4573

Reference manual is unclear about the purpose of REFERENCEDCOLUMNS in SYSTRIGGERS table.

    Details

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

      Description

      The Reference manual (applies to all the releases) says following for REFERENCEDCOLUMNS in SYSTRIGGERS system table
      "descriptor of the columns referenced by UPDATE triggers".

      This is confusing because it does not indicate whether the columns referenced by the triggering statement or the columns referenced by the trigger action. My suggestion is as follows but please feel free to reword it if required.
      "descriptor of columns on which UPDATE trigger has been defined"

      1. DERBY-4573.diff
        0.6 kB
        Kim Haase
      2. rrefsistabs79888.html
        11 kB
        Kim Haase
      3. DERBY-4573-2.diff
        0.6 kB
        Kim Haase
      4. rrefsistabs79888.html
        11 kB
        Kim Haase

        Issue Links

          Activity

          Hide
          Kim Haase added a comment -

          If/when I file a second patch for DERBY-4691, I should fix this too.

          Show
          Kim Haase added a comment - If/when I file a second patch for DERBY-4691 , I should fix this too.
          Hide
          Kim Haase added a comment -

          Attaching DERBY-4573.diff and the modified rrefsistabs79888.html topic.

          I wasn't sure whether to say

          "Descriptor of the columns on which an UPDATE trigger has been defined"

          or

          "Descriptor of the columns on which UPDATE triggers have been defined"

          I think they both mean the same thing, essentially. I went with the first one. Hope that's okay.

          Show
          Kim Haase added a comment - Attaching DERBY-4573 .diff and the modified rrefsistabs79888.html topic. I wasn't sure whether to say "Descriptor of the columns on which an UPDATE trigger has been defined" or "Descriptor of the columns on which UPDATE triggers have been defined" I think they both mean the same thing, essentially. I went with the first one. Hope that's okay.
          Hide
          Dag H. Wanvik added a comment -

          Hi Kim, I think you made the right choice, since this information pertains to exactly one trigger (or none).

          Even more precise wording might perhaps be:

          If this trigger is an update trigger (i.e. the EVENT column contains 'U'), this is the descriptor of the columns to be updated.

          Show
          Dag H. Wanvik added a comment - Hi Kim, I think you made the right choice, since this information pertains to exactly one trigger (or none). Even more precise wording might perhaps be: If this trigger is an update trigger (i.e. the EVENT column contains 'U'), this is the descriptor of the columns to be updated.
          Hide
          Kim Haase added a comment -

          Thanks for that clarification, Dag! I'm attaching a revised patch, DERBY-4573-2.diff, and new version of the topic, with your change. I modified the word order to be consistent with the other items in that column of the table – hope that's okay.

          Show
          Kim Haase added a comment - Thanks for that clarification, Dag! I'm attaching a revised patch, DERBY-4573 -2.diff, and new version of the topic, with your change. I modified the word order to be consistent with the other items in that column of the table – hope that's okay.
          Hide
          Kim Haase added a comment -

          In the absence of further comment, I'm assuming the revision is okay. Thanks again, Dag.

          Committed patch DERBY-4573-2.diff to documentation trunk at revision 1060404.

          Show
          Kim Haase added a comment - In the absence of further comment, I'm assuming the revision is okay. Thanks again, Dag. Committed patch DERBY-4573 -2.diff to documentation trunk at revision 1060404.
          Hide
          Kim Haase added a comment -

          Change has appeared in latest alpha docs, so closing.

          Show
          Kim Haase added a comment - Change has appeared in latest alpha docs, so closing.

            People

            • Assignee:
              Kim Haase
              Reporter:
              Mamta A. Satoor
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development