Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 2.1.0
    • 2.1.0
    • Sentry
    • None

    Description

      Add e2e tests to verify owner privileges

      Attachments

        1. SENTRY-2247.001.patch
          12 kB
          Na Li
        2. SENTRY-2247.002.patch
          12 kB
          Na Li
        3. SENTRY-2247.003.patch
          19 kB
          Na Li
        4. SENTRY-2247.004.patch
          25 kB
          Na Li
        5. SENTRY-2247.005.patch
          31 kB
          Na Li

        Issue Links

          Activity

            hadoopqa Hadoop QA added a comment -

            Here are the results of testing the latest attachment
            https://issues.apache.org/jira/secure/attachment/12930055/SENTRY-2247.001.patch against master.

            Overall: -1 due to an error

            ERROR: failed to build with patch (exit code 1)

            Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3946/console

            This message is automatically generated.

            hadoopqa Hadoop QA added a comment - Here are the results of testing the latest attachment https://issues.apache.org/jira/secure/attachment/12930055/SENTRY-2247.001.patch against master. Overall: -1 due to an error ERROR: failed to build with patch (exit code 1) Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3946/console This message is automatically generated.
            hadoopqa Hadoop QA added a comment -

            Here are the results of testing the latest attachment
            https://issues.apache.org/jira/secure/attachment/12930058/SENTRY-2247.002.patch against master.

            Overall: -1 due to 2 errors

            ERROR: mvn test exited 1
            ERROR: Failed: org.apache.sentry.api.service.thrift.TestSentryPolicyStoreProcessor

            Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3947/console

            This message is automatically generated.

            hadoopqa Hadoop QA added a comment - Here are the results of testing the latest attachment https://issues.apache.org/jira/secure/attachment/12930058/SENTRY-2247.002.patch against master. Overall: -1 due to 2 errors ERROR: mvn test exited 1 ERROR: Failed: org.apache.sentry.api.service.thrift.TestSentryPolicyStoreProcessor Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3947/console This message is automatically generated.
            hadoopqa Hadoop QA added a comment -

            Here are the results of testing the latest attachment
            https://issues.apache.org/jira/secure/attachment/12930189/SENTRY-2247.003.patch against master.

            Overall: -1 due to 4 errors

            ERROR: mvn test exited 1
            ERROR: Failed: org.apache.sentry.tests.e2e.dbprovider.TestOwnerPrivilegesWithGrantOption
            ERROR: Failed: org.apache.sentry.tests.e2e.dbprovider.TestOwnerPrivilegesWithGrantOption
            ERROR: Failed: org.apache.sentry.tests.e2e.dbprovider.TestOwnerPrivilegesWithGrantOption

            Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3950/console

            This message is automatically generated.

            hadoopqa Hadoop QA added a comment - Here are the results of testing the latest attachment https://issues.apache.org/jira/secure/attachment/12930189/SENTRY-2247.003.patch against master. Overall: -1 due to 4 errors ERROR: mvn test exited 1 ERROR: Failed: org.apache.sentry.tests.e2e.dbprovider.TestOwnerPrivilegesWithGrantOption ERROR: Failed: org.apache.sentry.tests.e2e.dbprovider.TestOwnerPrivilegesWithGrantOption ERROR: Failed: org.apache.sentry.tests.e2e.dbprovider.TestOwnerPrivilegesWithGrantOption Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3950/console This message is automatically generated.
            hadoopqa Hadoop QA added a comment -

            Here are the results of testing the latest attachment
            https://issues.apache.org/jira/secure/attachment/12930407/SENTRY-2247.004.patch against master.

            Overall: +1 all checks pass

            SUCCESS: all tests passed

            Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3951/console

            This message is automatically generated.

            hadoopqa Hadoop QA added a comment - Here are the results of testing the latest attachment https://issues.apache.org/jira/secure/attachment/12930407/SENTRY-2247.004.patch against master. Overall: +1 all checks pass SUCCESS: all tests passed Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3951/console This message is automatically generated.
            hadoopqa Hadoop QA added a comment -

            Here are the results of testing the latest attachment
            https://issues.apache.org/jira/secure/attachment/12930444/SENTRY-2247.005.patch against master.

            Overall: +1 all checks pass

            SUCCESS: all tests passed

            Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3952/console

            This message is automatically generated.

            hadoopqa Hadoop QA added a comment - Here are the results of testing the latest attachment https://issues.apache.org/jira/secure/attachment/12930444/SENTRY-2247.005.patch against master. Overall: +1 all checks pass SUCCESS: all tests passed Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/3952/console This message is automatically generated.
            linaataustin Na Li added a comment -

            This update no only adds e2e tests for owner privilege, it also fixes the following issues, so those tests can be enabled and pass

            • The server name of owner privilege was not set at server side
            • When dropping privilege, if a privilege has no role, it is not dropped
            • When dropping privilege, if it has grant option being true, it is not dropped.
            • When renaming table, if its privilege has grant option being true, it is not transferred to new table
            linaataustin Na Li added a comment - This update no only adds e2e tests for owner privilege, it also fixes the following issues, so those tests can be enabled and pass The server name of owner privilege was not set at server side When dropping privilege, if a privilege has no role, it is not dropped When dropping privilege, if it has grant option being true, it is not dropped. When renaming table, if its privilege has grant option being true, it is not transferred to new table

            People

              linaataustin Na Li
              kkalyan Krishna Kalyan
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: