Re: Should we exclude all Hadoop JARS from the Sentry package?

2018-10-26 Thread Sergio Pena
Yes On Fri, Oct 26, 2018, 3:28 PM Kalyan Kumar Kalvagadda wrote: > Sergio, > > Did you test it with HDFS synchronization? That's when sentry namenode > plug-in is used. > > > > *Thanks,Kalyan Kumar Kalvagadda* | Software Engineer > t. (469) 279- <00>5732 > cloudera.com

Re: Should we exclude all Hadoop JARS from the Sentry package?

2018-10-26 Thread Kalyan Kumar Kalvagadda
Sergio, Did you test it with HDFS synchronization? That's when sentry namenode plug-in is used. *Thanks,Kalyan Kumar Kalvagadda* | Software Engineer t. (469) 279- <00>5732 cloudera.com [image: Cloudera] [image: Cloudera on Twitter

Re: Should we exclude all Hadoop JARS from the Sentry package?

2018-10-26 Thread Sergio Pena
Thanks Kalyan. I don't think we need the hadoop jars to be deployed to Sentry as it uses the ones available in the namenode classpath. Actually, I tested Sentry that depends on hadoop3 in a Hadoop2 environment without putting the hadoop jars in the Namenode classpath. That made me think that those

Re: Review Request 68926: SENTRY-2372: SentryStore should not implement grantOptionCheck

2018-10-26 Thread Vasili Zolotov via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68926/#review210102 --- Fix it, then Ship it! LGTM sentry-service/sentry-service-serv

Re: Should we exclude all Hadoop JARS from the Sentry package?

2018-10-26 Thread Kalyan Kumar Kalvagadda
Sergio, Some of these hadoop jars are needed for sentry namenode plug-in. Sentry currently uses "compile" as dependency scope for all the hadoop dependencies. If we use "provided" as scope, sentry will not be distributing this jars but it expects they are made available are runtime. we could stop

Re: Should we exclude all Hadoop JARS from the Sentry package?

2018-10-26 Thread Sergio Pena
sentry-dist/target/apache-sentry-2.2.0-SNAPSHOT-bin/apache-sentry-2.2.0-SNAPSHOT-bin/lib/hadoop-annotations-3.1.1.jar sentry-dist/target/apache-sentry-2.2.0-SNAPSHOT-bin/apache-sentry-2.2.0-SNAPSHOT-bin/lib/hadoop-auth-3.1.1.jar sentry-dist/target/apache-sentry-2.2.0-SNAPSHOT-bin/apache-sentry-2.2.

Re: Should we exclude all Hadoop JARS from the Sentry package?

2018-10-26 Thread Stephen Moist
Which Hadoop jars are they? > On Oct 26, 2018, at 9:43 AM, Sergio Pena > wrote: > > I see several hadoop jars in the final Sentry package. > > Do we really need those jars be available in the final Sentry .tar.gz? > Should we exclude them? > > - Sergio

Should we exclude all Hadoop JARS from the Sentry package?

2018-10-26 Thread Sergio Pena
I see several hadoop jars in the final Sentry package. Do we really need those jars be available in the final Sentry .tar.gz? Should we exclude them? - Sergio