Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15796 )
Change subject: IMPALA-9649: Exclude shiro* and add to banned dependency maven plugin ...................................................................... IMPALA-9649: Exclude shiro* and add to banned dependency maven plugin The earlier attempt to exclude the shiro-core and shiro-crypto-cipher jars from fe/pom.xml failed to find all instances, and security scans picked them up again. This patch also excludes the jar from the following: - sentry-core-common - sentry-provider-cache - sentry-provider-db - sentry-provider-file Furthermore, to avoid compilation errors related to the absense of shiro, it was necessary to remove the TestSentryResourceAuthorizationProvider class, and any tests that referenced it. Since Sentry is not being used any longer, this shouldn't be an issue. Tested by running build, which didn't fail from banned dependency plugin, as well running the standard set of tests on jenkins.impala.io. Change-Id: I9f9994bf81c1d2e025a03925e8eccb147c34d66e Reviewed-on: http://gerrit.cloudera.org:8080/15796 Reviewed-by: Joe McDonnell <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/pom.xml M fe/src/test/java/org/apache/impala/authorization/AuthorizationTest.java D fe/src/test/java/org/apache/impala/testutil/TestSentryResourceAuthorizationProvider.java M tests/authorization/test_authorization.py M tests/authorization/test_grant_revoke.py D tests/authorization/test_owner_privileges.py M tests/authorization/test_sentry.py D tests/authorization/test_show_grant.py 8 files changed, 43 insertions(+), 1,004 deletions(-) Approvals: Joe McDonnell: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/15796 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9f9994bf81c1d2e025a03925e8eccb147c34d66e Gerrit-Change-Number: 15796 Gerrit-PatchSet: 10 Gerrit-Owner: David Knupp <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Dinesh Garg (430) Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
