New release branch 2.0.0

2017-11-27 Thread Kalyan Kumar Kalvagadda
I've just created new release branch for upcoming 2.0.0 release. Please continue committing to master branch as usual. I'll cherry-pick commits to branch-2.0.0 on need basis. -Kalyan

Re: Review Request 64002: SENTRY-2068: Disable HTTP TRACE method from the Sentry Web Server

2017-11-27 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64002/#review191899 --- Ship it! Ship It! - Na Li On Nov. 22, 2017, 3:19 p.m.,

Re: New release branch 2.0.0

2017-11-27 Thread Kalyan Kumar Kalvagadda
Sure, I will change the branch to sentry-2.0 Thanks, Kalyan -Kalyan On Mon, Nov 27, 2017 at 10:54 AM, Sergio Pena wrote: > That's old Kalyan. Why do you propose bringing back the old approach? The > new approach is used by other Apache components where bugfix

Re: New release branch 2.0.0

2017-11-27 Thread Sergio Pena
Great, thanks Kalyan. Btw, the branch should be named branch-2.0 as the Wiki suggests (git checkout -b branch-1.8). It is better to name a branch this way so we can commit bugfixes on it for future 2.0.x bugfix releases. On Mon, Nov 27, 2017 at 8:19 AM, Kalyan Kumar Kalvagadda <

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

2017-11-27 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63596/ --- (Updated Nov. 27, 2017, 6 p.m.) Review request for sentry, Alexander Kolbasov,

Re: New release branch 2.0.0

2017-11-27 Thread Kalyan Kumar Kalvagadda
Sergio, If you look at all the previous branches they are followed convention sentry-major.minor.patch. I tried to be in-line with them. -Kalyan On Mon, Nov 27, 2017 at 9:35 AM, Sergio Pena wrote: > Great, thanks Kalyan. > > Btw, the branch should be named branch-2.0

Re: New release branch 2.0.0

2017-11-27 Thread Sergio Pena
That's old Kalyan. Why do you propose bringing back the old approach? The new approach is used by other Apache components where bugfix releases are only tags. This reduces the release process and steps of bugfix releases. On Mon, Nov 27, 2017 at 10:36 AM, Kalyan Kumar Kalvagadda <

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

2017-11-27 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/ --- (Updated Nov. 27, 2017, 4:39 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

2017-11-27 Thread Arjun Mishra via Review Board
> On Nov. 27, 2017, 6:22 p.m., Na Li wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Lines 351 (patched) > > > > > > this should be debug level or

Review Request 64097: SENTRY-2072: log4j2 dependencies brought by Hive 2 are causing conflicts with Sentry log4j

2017-11-27 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64097/ --- Review request for sentry and Sergio Pena. Bugs: SENTRY-2072

Re: Review Request 63964: SENTRY-2049: Remove hive-authz2 profile from the sentry-dist module

2017-11-27 Thread Steve Moist via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63964/ --- (Updated Nov. 27, 2017, 9:36 p.m.) Review request for sentry. Bugs:

Re: Review Request 64097: SENTRY-2072: log4j2 dependencies brought by Hive 2 are causing conflicts with Sentry log4j

2017-11-27 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64097/#review191930 --- Ship it! I tested this patch in a cluster environment and the

Re: Review Request 64097: SENTRY-2072: log4j2 dependencies brought by Hive 2 are causing conflicts with Sentry log4j

2017-11-27 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64097/#review191933 --- Ship it! Ship It! - Na Li On Nov. 27, 2017, 8:16 p.m.,

Re: Review Request 64097: SENTRY-2072: log4j2 dependencies brought by Hive 2 are causing conflicts with Sentry log4j

2017-11-27 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64097/#review191954 --- Ship it! Ship It! - Sergio Pena On Nov. 27, 2017, 11:22

Re: Review Request 64097: SENTRY-2072: log4j2 dependencies brought by Hive 2 are causing conflicts with Sentry log4j

2017-11-27 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64097/ --- (Updated Nov. 27, 2017, 11:22 p.m.) Review request for sentry and Sergio Pena.