Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/ --- (Updated Dec. 1, 2017, 2:03 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/#review192500 --- Is there a way to exclude the Apache licensed jars from the LICENS

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread kalyan kumar kalvagadda via Review Board
> On Dec. 1, 2017, 3:58 p.m., Colm O hEigeartaigh wrote: > > Is there a way to exclude the Apache licensed jars from the LICENSE.txt? > > Is it possible to remove the Stax-API bit for GPL? There are two sections, > > one GPL and one CDDL. > > You could also change the root LICENSE.txt in our sou

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/ --- (Updated Dec. 1, 2017, 5:27 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/#review192523 --- Is there any documentation about how to maintain this new licensin

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/ --- (Updated Dec. 1, 2017, 6:16 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/ --- (Updated Dec. 1, 2017, 8:50 p.m.) Review request for sentry, Alexander Kolbasov

Review Request 64259: SENTRY-1572 SentryMain() shouldn't dynamically load tool class

2017-12-01 Thread Xinran Tinney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64259/ --- Review request for sentry, Alexander Kolbasov, Arjun Mishra, kalyan kumar kalvag

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/#review192552 --- README_licesence.me Lines 1 (patched)

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/ --- (Updated Dec. 1, 2017, 9:29 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/#review192575 --- Ship it! Ship It! - Sergio Pena On Dec. 1, 2017, 9:29 p.m.,

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/#review192558 --- LICENSE.txt Line 214 (original)

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/ --- (Updated Dec. 1, 2017, 11:12 p.m.) Review request for sentry, Alexander Kolbaso

Review Request 64273: SENTRY-2084: Exclude javax.jms:jms from sentry distribution

2017-12-01 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64273/ --- Review request for sentry, Alexander Kolbasov and Sergio Pena. Bugs: SENTRY-208

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64241/#review192594 --- Ship it! Ship It! - Alexander Kolbasov On Dec. 1, 2017, 11:1

Re: Review Request 64273: SENTRY-2084: Exclude javax.jms:jms from sentry distribution

2017-12-01 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64273/#review192595 --- Ship it! Ship It! - Sergio Pena On Dec. 1, 2017, 11:19 p.m.,

Re: Review Request 64273: SENTRY-2084: Exclude javax.jms:jms from sentry distribution

2017-12-01 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64273/#review192596 --- Fix it, then Ship it! Ship It! pom.xml Lines 734 (patched)

Re: Review Request 64241: SENTRY-2081: Update the LICENSE.txt with the license information of distributed jars

2017-12-01 Thread kalyan kumar kalvagadda via Review Board
> On Dec. 1, 2017, 10:28 p.m., Alexander Kolbasov wrote: > > LICENSE.txt > > Line 214 (original) > > > > > > Should theLICENSE.txt include copyrights as well or not? Looks like you > > are removing some copyrights.

[VOTE] Release Sentry 2.0.0 RC2

2017-12-01 Thread Kalyan Kumar Kalvagadda
This is the release of Apache Sentry, version 2.0.0. It fixes the following issues: *https://issues.apache.org/jira/projects/SENTRY/versions/12341081 * Maven artifacts are available here:https://repository.apache.org/content/reposi

Re: [VOTE] Release Sentry 2.0.0 RC2

2017-12-01 Thread Alexander Kolbasov
Would you mind submitting the vote on Monday? This will give people 72 working hours to look at it. Thanks, - Alex. > On Dec 1, 2017, at 11:11 PM, Kalyan Kumar Kalvagadda > wrote: > > This is the release of Apache Sentry, version 2.0.0. > > It fixes the following issues: > *https://issues.a