Re: Review Request 63775: SENTRY-1543 Rename methods to be clearer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63775/#review191083 --- The subject of the review board must match the JIRA subject. You

Re: Shading of conflicting dependencies

+1 This gives sentry more flexibility without impacting other components using sentry. On Wed, Nov 15, 2017 at 12:04 PM, Sergio Pena wrote: > +1 > > I like the idea. It's hard to upgrade our libraries to newer ones when > other components break due to Sentry being a

Re: Shading of conflicting dependencies

+1 It's good approach but I have a question/concern. Is the proposal to shade is for some specific jars OR to shade all the third party jars? If proposal is shade all the third-party jars then if would impact the run time memory usage as all the classes from the third-party jars would be loaded

Re: Shading of conflicting dependencies

+1 I like the idea. It's hard to upgrade our libraries to newer ones when other components break due to Sentry being a plugin. I was once thought of using different jars versions per plugin (i.e. guava11 on hdfs, guava14 on hive and the server, etc), but that is too much to do and not good. I

Re: Review Request 63668: SENTRY-2038 - Some ShellCommand improvements

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63668/#review191082 --- Ship it! Ship It! - Sergio Pena On Nov. 15, 2017, 11:04

Re: Shading of conflicting dependencies

Agreed, this would be a very useful thing to do. I remember spending a lot of time trying to make Sentry work with DataNucleus 4 - the problem was that e2e tests combine Sentry with Hive in the same JVM and this created a conflict on the DataNucleus libraries and test failures. Looking at the

Re: Shading of conflicting dependencies

HBase does have a ensure-jars-have-correct-contents.sh check that they run which we could do something similar to. On Wed, Nov 15, 2017 at 3:30 PM Stephen Moist wrote: > Is there any way to add a check to the maven build to make sure that the > classes being imported are the

Re: Shading of conflicting dependencies

@Moist It can be found here https://github.com/apache/hbase/blob/master/hbase-shaded/hbase-shaded-check-invariants/src/test/resources/ensure-jars-have-correct-contents.sh On Wed, Nov 15, 2017 at 3:38 PM Brian Towles wrote: > HBase does have a

Re: Review Request 63775: SENTRY-1543 dropOrRenamePrivilegeForAllRoles() has confusing code

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63775/ --- (Updated Nov. 15, 2017, 9:47 p.m.) Review request for sentry. Summary

Re: Shading of conflicting dependencies

Is there any way to add a check to the maven build to make sure that the classes being imported are the shaded ones? > On Nov 15, 2017, at 3:02 PM, Brian Towles wrote: > > I can see that. I can look into that as well. It would make sense for the > "Sentry as a Library"

Re: Review Request 63402: SENTRY-2021 MR session ACLs in Hive binding does not handle all types of ACLs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63402/ --- (Updated Nov. 15, 2017, 9:02 p.m.) Review request for sentry. Changes

Re: Shading of conflicting dependencies

Brian, There are cases where shading is useful for server as well - mostly for e2e tests which run server, HDFS, Hive all in the same JVM. On Wed, Nov 15, 2017 at 12:47 PM, Brian Towles wrote: > So there would be two different levels of shading. > > The first would be the

Re: Shading of conflicting dependencies

I can see that. I can look into that as well. It would make sense for the "Sentry as a Library" you talked about as well On Wed, Nov 15, 2017 at 2:56 PM Alexander Kolbasov wrote: > Brian, There are cases where shading is useful for server as well - mostly > for e2e tests

Re: Review Request 63402: SENTRY-2021 MR session ACLs in Hive binding does not handle all types of ACLs

> On Nov. 14, 2017, 8:19 p.m., Sergio Pena wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingSessionHook.java > > Lines 116-117 (original), 116-122 (patched) > >

Review Request 63856: SENTRY-2048: Bump Hive version to 2.3.2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63856/ --- Review request for sentry, Alexander Kolbasov, Colm O hEigeartaigh, and kalyan

Re: Review Request 63668: SENTRY-2038 - Some ShellCommand improvements

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63668/ --- (Updated Nov. 15, 2017, 11:04 a.m.) Review request for sentry. Changes

Shading of conflicting dependencies

Howdy all, An issue that keeps coming up seems to be the conflict of dependency versions between Sentry and the components it is plugging into. A current example of this impact is Google Guava with hive2 using v14 and Impala using v11 while Sentry needs to have at least v14 in order to fix some