Accumulo
  1. Accumulo
  2. ACCUMULO-2096

randomwalk AlterTable uses table name instead of ID

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.6.0
    • Component/s: test
    • Labels:

      Description

      RW test uses the table name instead of table ID at org.apache.accumulo.test.randomwalk.security.AlterTable.visit(AlterTable.java:40)

        Activity

        Hide
        ASF subversion and git services added a comment -

        Commit 9f59c0022777fc86d0b269d064bcc898b27c9b23 in branch refs/heads/1.6.0-SNAPSHOT from Josh Elser
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9f59c00 ]

        ACCUMULO-2096 Clean up the security RW tests.

        Ensure that tableID is used where necessary and ensure that the correct SecurityErrorCode is used when using the
        internal method calls (that throw ThriftSecurityException and not AccumuloSecurityException) so the assertions work
        correctly.

        Show
        ASF subversion and git services added a comment - Commit 9f59c0022777fc86d0b269d064bcc898b27c9b23 in branch refs/heads/1.6.0-SNAPSHOT from Josh Elser [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9f59c00 ] ACCUMULO-2096 Clean up the security RW tests. Ensure that tableID is used where necessary and ensure that the correct SecurityErrorCode is used when using the internal method calls (that throw ThriftSecurityException and not AccumuloSecurityException) so the assertions work correctly.
        Hide
        ASF subversion and git services added a comment -

        Commit 9f59c0022777fc86d0b269d064bcc898b27c9b23 in branch refs/heads/master from Josh Elser
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9f59c00 ]

        ACCUMULO-2096 Clean up the security RW tests.

        Ensure that tableID is used where necessary and ensure that the correct SecurityErrorCode is used when using the
        internal method calls (that throw ThriftSecurityException and not AccumuloSecurityException) so the assertions work
        correctly.

        Show
        ASF subversion and git services added a comment - Commit 9f59c0022777fc86d0b269d064bcc898b27c9b23 in branch refs/heads/master from Josh Elser [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=9f59c00 ] ACCUMULO-2096 Clean up the security RW tests. Ensure that tableID is used where necessary and ensure that the correct SecurityErrorCode is used when using the internal method calls (that throw ThriftSecurityException and not AccumuloSecurityException) so the assertions work correctly.
        Hide
        Christopher Tubbs added a comment -

        Renamed subject, because I don't know what "RW" is at first glance, and this is an unnecessary barrier to addressing the issue.

        Show
        Christopher Tubbs added a comment - Renamed subject, because I don't know what "RW" is at first glance, and this is an unnecessary barrier to addressing the issue.
        Hide
        John Vines added a comment -

        I disagree with this ticket. The state information for the security randomwalk was pegged on tablenames to prevent race conditions with the tableid mapping from updating in zookeeper on the walker's side. The only real conditions where this could be considered an issue is renameTable, but that updates the table information

        Show
        John Vines added a comment - I disagree with this ticket. The state information for the security randomwalk was pegged on tablenames to prevent race conditions with the tableid mapping from updating in zookeeper on the walker's side. The only real conditions where this could be considered an issue is renameTable, but that updates the table information
        Hide
        ASF subversion and git services added a comment -

        Commit 90e1f22a1833baad0f986f2100c6ecb5662a8a8f in branch refs/heads/1.6.0-SNAPSHOT from John Vines
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=90e1f22 ]

        ACCUMULO-1479 ACCUMULO-2086 ACCUMULO-2096 refactoring namespace translation out of security operations to fix security randomwalk

        Show
        ASF subversion and git services added a comment - Commit 90e1f22a1833baad0f986f2100c6ecb5662a8a8f in branch refs/heads/1.6.0-SNAPSHOT from John Vines [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=90e1f22 ] ACCUMULO-1479 ACCUMULO-2086 ACCUMULO-2096 refactoring namespace translation out of security operations to fix security randomwalk
        Hide
        ASF subversion and git services added a comment -

        Commit 3d8a1161e88b613e48b23e16cca12699d0ba0bf7 in branch refs/heads/1.6.0-SNAPSHOT from John Vines
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=3d8a116 ]

        ACCUMULO-2086 ACCUMULO-2096 ACCUMULO-2286 Some more initial namespace support

        Show
        ASF subversion and git services added a comment - Commit 3d8a1161e88b613e48b23e16cca12699d0ba0bf7 in branch refs/heads/1.6.0-SNAPSHOT from John Vines [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=3d8a116 ] ACCUMULO-2086 ACCUMULO-2096 ACCUMULO-2286 Some more initial namespace support
        Hide
        ASF subversion and git services added a comment -

        Commit 90e1f22a1833baad0f986f2100c6ecb5662a8a8f in branch refs/heads/master from John Vines
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=90e1f22 ]

        ACCUMULO-1479 ACCUMULO-2086 ACCUMULO-2096 refactoring namespace translation out of security operations to fix security randomwalk

        Show
        ASF subversion and git services added a comment - Commit 90e1f22a1833baad0f986f2100c6ecb5662a8a8f in branch refs/heads/master from John Vines [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=90e1f22 ] ACCUMULO-1479 ACCUMULO-2086 ACCUMULO-2096 refactoring namespace translation out of security operations to fix security randomwalk
        Hide
        ASF subversion and git services added a comment -

        Commit 3d8a1161e88b613e48b23e16cca12699d0ba0bf7 in branch refs/heads/master from John Vines
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=3d8a116 ]

        ACCUMULO-2086 ACCUMULO-2096 ACCUMULO-2286 Some more initial namespace support

        Show
        ASF subversion and git services added a comment - Commit 3d8a1161e88b613e48b23e16cca12699d0ba0bf7 in branch refs/heads/master from John Vines [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=3d8a116 ] ACCUMULO-2086 ACCUMULO-2096 ACCUMULO-2286 Some more initial namespace support
        Hide
        John Vines added a comment -

        Tests should be kosher now. Changes made in the previous commit were reverted.

        Show
        John Vines added a comment - Tests should be kosher now. Changes made in the previous commit were reverted.

          People

          • Assignee:
            John Vines
            Reporter:
            Josh Elser
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development