Derby
  1. Derby
  2. DERBY-5822

Document the new SQL standard behavior of the SPACE_TABLE vti

    Details

    • Urgency:
      Normal
    • Bug behavior facts:
      Deviation from standard

      Description

      We will want to improve the Reference Guide section on the SPACE_TABLE vti after we commit the derby-5554-02-aa-0argConstructor.diff attached to DERBY-5554. We will want to make the following changes:

      1) Describe the new TABLEID column which was added to SPACE_TABLE:

      Column Name: TABLEID
      Type: CHAR
      Length: 36
      Nullable: False
      Contents: The id of the table which the conglomerate belongs to.

      2) Describe the additional no-arg constructor for this vti. There are now 3 ways to construct this vti:

      i) No args. This is the new constructor which needs to be described. When the vti is invoked with no arguments, it retrieves space information for all tables and indexes in the database.

      ii) One argument. This constructor is already described. If the vti is invoked with one string argument, then that argument is the name of a table in the current schema.

      iii) Two arguments. This constructor is already described also. When the vti is invoked with two string arguments, the first argument is a schema name and the second argument is a table name.

      3) Replace the first SPACE_TABLE example. That example uses Derby-specific syntax which falls outside the SQL Standard. Showcasing non-standard syntax encourages users to try out other non-standard syntax, giving rise to bugs like DERBY-5554. We should showcase the following standard syntax instead:

      select t2.*
      from
      sys.systables systabs,
      table (syscs_diag.space_table()) as t2
      where systabs.tabletype = 'T'
      and systabs.tableid = t2.tableid;

      1. rrefsyscsdiagtables.html
        24 kB
        Kim Haase
      2. DERBY-5822.diff
        3 kB
        Kim Haase

        Issue Links

          Activity

          Hide
          Kim Haase added a comment -

          Closing, since changes have appeared in Latest Alpha Manuals (as well as 10.9 and 10.8).

          Show
          Kim Haase added a comment - Closing, since changes have appeared in Latest Alpha Manuals (as well as 10.9 and 10.8).
          Hide
          Kim Haase added a comment -

          Thanks very much, Rick.

          Merged DERBY-5822.diff to 10.9 doc branch at revision 1355150.

          Merged DERBY-5822.diff to 10.8 doc branch at revision 1355154.

          Show
          Kim Haase added a comment - Thanks very much, Rick. Merged DERBY-5822 .diff to 10.9 doc branch at revision 1355150. Merged DERBY-5822 .diff to 10.8 doc branch at revision 1355154.
          Hide
          Rick Hillegas added a comment -

          Thanks, Kim. Please port the doc fix to 10.9 and 10.8. I have added a comment to DERBY-5554, explaining why I have left that issue open. Thanks.

          Show
          Rick Hillegas added a comment - Thanks, Kim. Please port the doc fix to 10.9 and 10.8. I have added a comment to DERBY-5554 , explaining why I have left that issue open. Thanks.
          Hide
          Kim Haase added a comment -

          Thanks, Rick! Committed patch DERBY-5822.diff to documentation trunk at revision 1355117.

          It appears that the fix for DERBY-5554 has been ported to the 10.9 and 10.8 branches, but the issue has not yet been resolved. Is it okay to port the doc fix to the branches as well?

          Show
          Kim Haase added a comment - Thanks, Rick! Committed patch DERBY-5822 .diff to documentation trunk at revision 1355117. It appears that the fix for DERBY-5554 has been ported to the 10.9 and 10.8 branches, but the issue has not yet been resolved. Is it okay to port the doc fix to the branches as well?
          Hide
          Rick Hillegas added a comment -

          Thanks, Kim. The patch looks good to me. +1

          Show
          Rick Hillegas added a comment - Thanks, Kim. The patch looks good to me. +1
          Hide
          Kim Haase added a comment -

          Attaching DERBY-5822.diff and rrefsyscsdiagtables.html, with changes to rrefsyscsdiagtables.dita ("SYSCS_DIAG diagnostic tables and functions").

          Thanks for the very clear description, Rick! Please let me know what changes are needed.

          Show
          Kim Haase added a comment - Attaching DERBY-5822 .diff and rrefsyscsdiagtables.html, with changes to rrefsyscsdiagtables.dita ("SYSCS_DIAG diagnostic tables and functions"). Thanks for the very clear description, Rick! Please let me know what changes are needed.
          Hide
          Rick Hillegas added a comment -

          Hi Kim. Yes, we should fix the documentation in 10.9 and 10.8 after I port the fixes to those branches.

          Hi Kathey. Yes, the old syntax still works. We just want to steer users toward standard syntax which will work in more cases.

          Show
          Rick Hillegas added a comment - Hi Kim. Yes, we should fix the documentation in 10.9 and 10.8 after I port the fixes to those branches. Hi Kathey. Yes, the old syntax still works. We just want to steer users toward standard syntax which will work in more cases.
          Hide
          Kathey Marsden added a comment -

          Am I correct that the old example spacetable syntax will still execute but is just needs to be discouraged?

          Show
          Kathey Marsden added a comment - Am I correct that the old example spacetable syntax will still execute but is just needs to be discouraged?
          Hide
          Kim Haase added a comment -

          If DERBY-5554 is ported to 10.9 and 10.8, I am assuming the revised documentation should also be committed to those branches when it is completed. Let me know if this assumption is unfounded.

          Show
          Kim Haase added a comment - If DERBY-5554 is ported to 10.9 and 10.8, I am assuming the revised documentation should also be committed to those branches when it is completed. Let me know if this assumption is unfounded.

            People

            • Assignee:
              Kim Haase
              Reporter:
              Rick Hillegas
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development