Re: Proposal to release Sentry 2.0.0

2017-11-22 Thread Kalyan Kumar Kalvagadda
I have started the release process. As initial step I have moved all the unresolved jira's to 2.1.0 release except for SENTRY-2062 and SENTRY-1812. Please let me know if you want your changes to be part of sentry 2.0.0. I will be cutting branch for sentry 2.0.0 later today. -Kalyan On Mon, Oct 1

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

2017-11-22 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64002/ --- (Updated Nov. 22, 2017, 3:19 p.m.) Review request for sentry. Changes ---

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

2017-11-22 Thread Sergio Pena via Review Board
> On Nov. 21, 2017, 9:06 p.m., Na Li wrote: > > what test have you done for this update? I added a new patch with the test methods. - Sergio --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64

Re: Review Request 63424: SENTRY-2024: Specify Char Set for AUTHZ_OBJ_NAME

2017-11-22 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63424/#review191739 --- The index for AUTHZ_SNAPSHOT_ID is ok to keep it. If we have multi

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

2017-11-22 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63402/#review191743 --- Ship it! Ship It! - Sergio Pena On Nov. 15, 2017, 9:13 p.m.,

Re: Review Request 63985: SENTRY-2056: Display test-patch.py output on the standard console to see progress on Jenkins

2017-11-22 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63985/#review191744 --- || visible in terminal || visible in file || existing Syntax

Re: Review Request 63985: SENTRY-2056: Display test-patch.py output on the standard console to see progress on Jenkins

2017-11-22 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63985/ --- (Updated Nov. 22, 2017, 5:29 p.m.) Review request for sentry. Bugs: sentry-20

Re: Review Request 63985: SENTRY-2056: Display test-patch.py output on the standard console to see progress on Jenkins

2017-11-22 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63985/#review191746 --- Ship it! looks good - kalyan kumar kalvagadda On Nov. 22, 20

Re: Review Request 63424: SENTRY-2024: Specify Char Set for AUTHZ_OBJ_NAME

2017-11-22 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63424/ --- (Updated Nov. 22, 2017, 5:52 p.m.) Review request for sentry, Arjun Mishra, kal

Re: Review Request 63424: SENTRY-2024: Specify Char Set for AUTHZ_OBJ_NAME

2017-11-22 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63424/#review191750 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 63424: SENTRY-2024: Specify Char Set for AUTHZ_OBJ_NAME

2017-11-22 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63424/ --- (Updated Nov. 22, 2017, 6:29 p.m.) Review request for sentry, Arjun Mishra, kal

Re: Review Request 63424: SENTRY-2024: Specify Char Set for AUTHZ_OBJ_NAME

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

Avoiding merge commits to master.

2017-11-22 Thread Alexander Kolbasov
This is a reminder for developers to avoid merge commits to master. There might be cases where this is unavoidable, please send a note here if you think you MUST have a merge commit. Otherwise, please make an effort co collapse all intermediate changes/merges and create a single commit per changese

Review Request 64040: SENTRY-2070: TestSentryShellSolr tests are case sensitive to privileges

2017-11-22 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64040/ --- Review request for sentry, Hrishikesh Gadre, kalyan kumar kalvagadda, and Sergio

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

2017-11-22 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64002/#review191759 --- Ship it! Ship It! - Alexander Kolbasov On Nov. 22, 2017, 3:1

Re: Review Request 64040: SENTRY-2070: TestSentryShellSolr tests are case sensitive to privileges

2017-11-22 Thread Hrishikesh Gadre via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64040/#review191761 --- Although this change looks harmless, I am trying to understand the

Re: Review Request 63974: SENTRY-2062: Support the new Hive 2.3.2 DbNotificationListener

2017-11-22 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63974/#review191762 --- Ship it! Looks good. - kalyan kumar kalvagadda On Nov. 21, 2

Re: Proposal to release Sentry 2.0.0

2017-11-22 Thread Sergio Pena
Thanks Kalyan. Those JIRAs are committed now and the 2.0 release is not blocked anymore. You may want to cut the branch now to freeze the 2.0 version. - Sergio On Wed, Nov 22, 2017 at 9:13 AM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > I have started the release process. As initi