> On July 14, 2017, 1:08 p.m., Colm O hEigeartaigh wrote: > > I think this is not against the latest source, as I already fixed the test > > compilation issue in > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PrivilegeResultSet.java > > on that branch. > > kalyan kumar kalvagadda wrote: > Ohh right, I have created this patch two ago. I saw your commit > yesterday. I will update the patch and submitit again. > > Colm O hEigeartaigh wrote: > Great thanks! I also wonder if there's any point in keeping the separate > hive profiles, if they are both being built by default?
Both the profile are getting build but not tested. sentry-tests-hive-v2 will not be tested. - kalyan kumar ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60865/#review180521 ----------------------------------------------------------- On July 14, 2017, 1:02 p.m., kalyan kumar kalvagadda wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60865/ > ----------------------------------------------------------- > > (Updated July 14, 2017, 1:02 p.m.) > > > Review request for sentry and Sergio Pena. > > > 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 > ----- > > 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 > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/PrivilegeResultSet.java > 8818c4c > > > Diff: https://reviews.apache.org/r/60865/diff/1/ > > > Testing > ------- > > > Thanks, > > kalyan kumar kalvagadda > >
