Re: [VOTE] Release Sentry version 2.0.0

Hi Kalyan, Why do we vote on source + bin distributions that are copied to a local directory, when they are available in maven here? https://repository.apache.org/content/repositories/orgapachesentry-1005/org/apache/sentry/sentry-dist/2.0.0/ I found a minor problem in that some test

Re: [VOTE] Release Sentry version 2.0.0

Kalyan, Thank you for posting Release 2.0 candidate. What kind if testing would you recommend for us to perform to vote Yes/No? Can you post some recommendations, please? - Alex On Tue, Nov 28, 2017 at 4:38 PM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > This is the release of

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/ --- (Updated Nov. 29, 2017, 5:43 p.m.) Review request for sentry, kalyan kumar

Re: [VOTE] Release Sentry version 2.0.0

Agree with Colm. The licensing issues could be a problem for the official release, so we should address them. I've seen other components blocking releases due to licensing issues. On Wed, Nov 29, 2017 at 10:22 AM, Colm O hEigeartaigh wrote: > If you are happy with the

Re: Review Request 63596: SENTRY-1951 - Old SentryStore.retrieveFullPathsImage() should be removed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63596/ --- (Updated Nov. 29, 2017, 5:20 p.m.) Review request for sentry, Alexander

Re: Review Request 63596: SENTRY-1951 - Old SentryStore.retrieveFullPathsImage() should be removed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63596/ --- (Updated Nov. 29, 2017, 5:23 p.m.) Review request for sentry, Alexander

Re: Review Request 63596: SENTRY-1951 - Old SentryStore.retrieveFullPathsImage() should be removed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63596/ --- (Updated Nov. 29, 2017, 5:22 p.m.) Review request for sentry, Alexander

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/#review192133 ---

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

> On Nov. 29, 2017, 4:50 p.m., Sergio Pena wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Lines 251-252 (patched) > > > > > > This line is making me

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/ --- (Updated Nov. 29, 2017, 5:24 p.m.) Review request for sentry, kalyan kumar

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/ --- (Updated Nov. 29, 2017, 5:28 p.m.) Review request for sentry, kalyan kumar

Re: [VOTE] Release Sentry version 2.0.0

Sergio, I know that you tried to start sentry standalone, Did you see this issue? -Kalyan On Wed, Nov 29, 2017 at 12:16 PM, Alexander Kolbasov wrote: > I tried to start Sentry server on Mac and got a weird error . I am running > from the dist directory that I just built

Re: [VOTE] Release Sentry version 2.0.0

I'm trying to gather the license information for the dependencies. I'm trying to use Apache Maven Project Info Reports Plugin . Does anyone has a better suggestion? -Kalyan On Wed, Nov 29, 2017 at 12:32 PM, Sergio Pena

Re: [VOTE] Release Sentry version 2.0.0

I downloaded binary version that Kalyan posted and run it and got the same problem. Here is my java information: java version "1.8.0_101" Java(TM) SE Runtime Environment (build 1.8.0_101-b13) Java HotSpot(TM) 64-Bit Server VM (build 25.101-b13, mixed mode) The sentry process is still running

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/#review192140 ---

Review Request 64179: SENTRY-2079: Sentry HA leader monitor does not work due to a mix of curator versions in the classpath

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64179/ --- Review request for sentry. Bugs: sentry-2079

Re: [VOTE] Release Sentry version 2.0.0

I have gathered all license information for all the dependencies using license-maven-plugin. Here is the file attached. Is there a particular format I should be using to update the license file in the repo? -Kalyan On Wed, Nov 29, 2017 at 1:52 PM, Colm O hEigeartaigh

Re: Review Request 64179: SENTRY-2079: Sentry HA leader monitor does not work due to a mix of curator versions in the classpath

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64179/#review192228 --- sentry-provider/sentry-provider-db/pom.xml Lines 230 (patched)

Re: [VOTE] Release Sentry version 2.0.0

There is one more blocker for Sentry 2.0 that needs to be committed, can someone help me review this patch? https://issues.apache.org/jira/browse/SENTRY-2079 I am able to configure Sentry 2.0 without other issues now. I can also enable the Sentry Web UI. What problems do you guys have with the

Re: [VOTE] Release Sentry version 2.0.0

When I enable Web UI I have this exception that I posted here. It goes away when web UI is disabled. The exception is the same whether I run from the target directory in a branch or using downloaded bits. - Alex On Wed, Nov 29, 2017 at 7:10 PM, Sergio Pena wrote: >

Re: Review Request 64179: SENTRY-2079: Sentry HA leader monitor does not work due to a mix of curator versions in the classpath

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64179/#review192231 --- Ship it! Ship It! - Na Li On Nov. 29, 2017, 10:37 p.m.,

Re: Review Request 64179: SENTRY-2079: Sentry HA leader monitor does not work due to a mix of curator versions in the classpath

> On Nov. 30, 2017, 2:58 a.m., Na Li wrote: > > sentry-provider/sentry-provider-db/pom.xml > > Lines 230 (patched) > > > > > > do you need to add this in both root pom.xml line 635 and here? It's not necessary.

Re: [VOTE] Release Sentry version 2.0.0

Hi Kalyan, Anything Apache related you can immediately discount - so Hadoop, Hive, HBase, Solr, Zookeeper, etc. I would suggest just deleting those jars from the list and googling the remaining jars to see what the license is. Colm. On Wed, Nov 29, 2017 at 6:55 PM, Kalyan Kumar Kalvagadda <

Re: [VOTE] Release Sentry version 2.0.0

Could it be caused by the fact there are multiple Servlet API jars in lib? Colm. On Wed, Nov 29, 2017 at 7:04 PM, Alexander Kolbasov wrote: > I downloaded binary version that Kalyan posted and run it and got the same > problem. > > Here is my java information: > > java

Re: [VOTE] Release Sentry version 2.0.0

I tried to start Sentry server on Mac and got a weird error . I am running from the dist directory that I just built with mvn clean install on the 2.0.0 branch. 17/11/29 10:13:25 ERROR thrift.SentryService: Error starting server java.lang.SecurityException: class "javax.servlet.DispatcherType"'s

Re: [VOTE] Release Sentry version 2.0.0

I don't have that issue. I downloaded the 2.0 tar.gz that is already signed and install it on an Ubuntu 16.04 machine with Hive 2.3.2. It has been running in a single node mode. I could check HMS notifications being pulled and updated, and checking the permissions. On Wed, Nov 29, 2017 at 12:20

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

> On Nov. 29, 2017, 4:50 p.m., Sergio Pena wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Lines 251-252 (patched) > > > > > > This line is making me