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

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,

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

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,

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

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

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

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,

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 ---

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,

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,

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:

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

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

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

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: 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