----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60865/ -----------------------------------------------------------
(Updated July 14, 2017, 3:31 p.m.) Review request for sentry and Sergio Pena. Changes ------- Colm, Made changes as per your comments. Bugs: SENTRY-1834 https://issues.apache.org/jira/browse/SENTRY-1834 Repository: sentry Description ------- Planning to enable hive-auth2 profile in upstream sentry-ha-redesign branch. As we are supporting hive auth V2 in that branch we need to keep it up to date. Disabling it is making the Hive-Authv2 binding implementation stale. Here are some os the issue disabling it. 1. Hive BindingV2 Code is not fixed when Hive BindingV1 is fixed. 2. Hive BindingV2 will not be green. Fixing would be a bigger issue. If they are enabled contributors will make sure that related cpde for Hive-Authv2 is also fixed/updated properly. Diffs (updated) ----- pom.xml 2c109a4 sentry-binding/pom.xml 71fe47c sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java c63cf64 sentry-dist/pom.xml 2d7f57e sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java 1415297 Diff: https://reviews.apache.org/r/60865/diff/2/ Changes: https://reviews.apache.org/r/60865/diff/1-2/ Testing ------- Thanks, kalyan kumar kalvagadda