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

          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)?
          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.
          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.
          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.
          Hide
          Kathey Marsden added a comment -

          Reopen for backport

          Show
          Kathey Marsden added a comment - Reopen for backport
          Hide
          Kathey Marsden added a comment -

          Assigning to myself for backport

          Show
          Kathey Marsden added a comment - Assigning to myself for backport
          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

            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