Derby
  1. Derby
  2. DERBY-4051

The javadoc for SpaceTable refers to an alias that doesn't seem to work

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 10.5.1.1
    • Fix Version/s: 10.5.3.2, 10.6.1.0
    • Component/s: Javadoc
    • Labels:
      None
    • Issue & fix info:
      Newcomer

      Description

      The javadoc for the SpaceTable vti refers to an alias that doesn't seem to work:
      http://db.apache.org/derby/javadoc/engine/org/apache/derby/diag/SpaceTable.html

      "Alternatively, the table can be invoked through the system alias SpaceTable

      select * from new SPACETABLE('MYTABLE') t; "

      1. derby-4051.patch
        1 kB
        Tiago R. Espinha

        Issue Links

          Activity

          Kathey Marsden created issue -
          Hide
          Knut Anders Hatlen added a comment -

          I think we should remove the mentioning of the alias and add an example with SYSCS_DIAG.SPACE_TABLE (which is the preferred way to invoke this VTI, isn't it?).

          SELECT * FROM TABLE(SYSCS_DIAG.SPACE_TABLE('MYTABLE')) T;
          SELECT * FROM TABLE(SYSCS_DIAG.SPACE_TABLE('MYSCHEMA', 'MYTABLE')) T;

          Show
          Knut Anders Hatlen added a comment - I think we should remove the mentioning of the alias and add an example with SYSCS_DIAG.SPACE_TABLE (which is the preferred way to invoke this VTI, isn't it?). SELECT * FROM TABLE(SYSCS_DIAG.SPACE_TABLE('MYTABLE')) T; SELECT * FROM TABLE(SYSCS_DIAG.SPACE_TABLE('MYSCHEMA', 'MYTABLE')) T;
          Hide
          Tiago R. Espinha added a comment -

          Hmm, so how do we really go about this? Do we just change the javadoc to include the example with the SYSCS_DIAG or do we try to get the alias working again (assuming it actually did work at some point)?

          I am confident I can do the former, but the latter will probably require someone more experienced...

          Show
          Tiago R. Espinha added a comment - Hmm, so how do we really go about this? Do we just change the javadoc to include the example with the SYSCS_DIAG or do we try to get the alias working again (assuming it actually did work at some point)? I am confident I can do the former, but the latter will probably require someone more experienced...
          Hide
          Kathey Marsden added a comment -

          I think we want to change the examples to use they SYSCS_DIAG procedure.

          Show
          Kathey Marsden added a comment - I think we want to change the examples to use they SYSCS_DIAG procedure.
          Hide
          Tiago R. Espinha added a comment -

          Okay, I'll assign this to me then and get it fixed.

          Kathey, is it worth checking if this is also present in other versions (and if the alias also doesn't work on those)?

          Show
          Tiago R. Espinha added a comment - Okay, I'll assign this to me then and get it fixed. Kathey, is it worth checking if this is also present in other versions (and if the alias also doesn't work on those)?
          Tiago R. Espinha made changes -
          Field Original Value New Value
          Assignee Tiago R. Espinha [ espinha ]
          Hide
          Kathey Marsden added a comment -

          I think you can just focus on trunk since this is just a javadoc issue.

          Show
          Kathey Marsden added a comment - I think you can just focus on trunk since this is just a javadoc issue.
          Hide
          Tiago R. Espinha added a comment -

          Kathey has suggested that we not only remove the alias, but also that we change the examples using this method:
          select * from new org.apache.derby.diag.SpaceTable('MYTABLE') t;

          by ones using the SYSCS_DIAG.SPACE_TABLE, since that method has been deprecated.

          I think it makes sense. Does anyone have a different opinion or just additional thoughts on this?

          Show
          Tiago R. Espinha added a comment - Kathey has suggested that we not only remove the alias, but also that we change the examples using this method: select * from new org.apache.derby.diag.SpaceTable('MYTABLE') t; by ones using the SYSCS_DIAG.SPACE_TABLE, since that method has been deprecated. I think it makes sense. Does anyone have a different opinion or just additional thoughts on this?
          Hide
          Tiago R. Espinha added a comment -

          No one has opposed to the idea of changing the references so far. Since the old method has been deprecated and the new one should be used, I'll be fixing this.

          Show
          Tiago R. Espinha added a comment - No one has opposed to the idea of changing the references so far. Since the old method has been deprecated and the new one should be used, I'll be fixing this.
          Hide
          Tiago R. Espinha added a comment -

          Fixed the two references on top, removed the reference to the alias since that does no longer exist and also changed the reference below.

          Show
          Tiago R. Espinha added a comment - Fixed the two references on top, removed the reference to the alias since that does no longer exist and also changed the reference below.
          Tiago R. Espinha made changes -
          Attachment derby-4051.patch [ 12406044 ]
          Tiago R. Espinha made changes -
          Derby Info [Patch Available]
          Hide
          Knut Anders Hatlen added a comment -

          Thanks Tiago. I verified that the SQL examples worked and committed the patch with revision 767396.

          Show
          Knut Anders Hatlen added a comment - Thanks Tiago. I verified that the SQL examples worked and committed the patch with revision 767396.
          Knut Anders Hatlen made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Derby Info [Patch Available]
          Fix Version/s 10.6.0.0 [ 12313727 ]
          Resolution Fixed [ 1 ]
          Hide
          Tiago R. Espinha added a comment -

          Thank you Knut closing the issue now.

          Show
          Tiago R. Espinha added a comment - Thank you Knut closing the issue now.
          Tiago R. Espinha made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Myrna van Lunteren made changes -
          Affects Version/s 10.5.1.1 [ 12313771 ]
          Affects Version/s 10.5.0.0 [ 12313010 ]
          Dag H. Wanvik made changes -
          Issue & fix info [Newcomer]
          Kathey Marsden made changes -
          Link This issue is required by DERBY-4994 [ DERBY-4994 ]
          Hide
          Kathey Marsden added a comment -

          Reopen for backport

          Show
          Kathey Marsden added a comment - Reopen for backport
          Kathey Marsden made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Hide
          Kathey Marsden added a comment -

          Assigning to myself for backport

          Show
          Kathey Marsden added a comment - Assigning to myself for backport
          Kathey Marsden made changes -
          Assignee Tiago R. Espinha [ espinha ] Kathey Marsden [ kmarsden ]
          Hide
          Kathey Marsden added a comment -

          Reclosing issue after backport to 10.5

          Show
          Kathey Marsden added a comment - Reclosing issue after backport to 10.5
          Kathey Marsden made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Assignee Kathey Marsden [ kmarsden ] Tiago R. Espinha [ espinha ]
          Fix Version/s 10.5.3.2 [ 12315436 ]
          Resolution Fixed [ 1 ]
          Gavin made changes -
          Workflow jira [ 12452052 ] Default workflow, editable Closed status [ 12798862 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          74d 14h 57m 1 Knut Anders Hatlen 22/Apr/09 08:27
          Resolved Resolved Closed Closed
          16m 33s 1 Tiago R. Espinha 22/Apr/09 08:44
          Closed Closed Reopened Reopened
          651d 16h 1 Kathey Marsden 02/Feb/11 23:44
          Reopened Reopened Closed Closed
          4d 17h 37m 1 Kathey Marsden 07/Feb/11 17:22

            People

            • Assignee:
              Tiago R. Espinha
              Reporter:
              Kathey Marsden
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development