Review Request 65053: SENTRY-2120: cross-site scripting vulnerability in LogLevelServlet

2018-01-09 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65053/ --- Review request for sentry, Brian Towles, kalyan kumar kalvagadda, and Sergio Pen

Re: Review Request 64949: SENTRY-641 Add binding for lily hbase indexer

2018-01-09 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64949/#review195088 --- sentry-binding/sentry-binding-hbase-indexer/src/main/java/org/apa

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

2018-01-09 Thread Xinran Tinney
> On Jan. 9, 2018, 10:16 p.m., Sergio Pena wrote: > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/SentryMain.java > > Line 18 (original), 18 (patched) > > > > > > This package name does not correspo

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

2018-01-09 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64259/#review195083 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/Se

Re: Review Request 65046: SENTRY-1819 HMSFollower and friends do not belong in sentry.service.thrift

2018-01-09 Thread Xinran Tinney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65046/ --- (Updated Jan. 9, 2018, 9:45 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 65046: SENTRY-1819 HMSFollower and friends do not belong in sentry.service.thrift

2018-01-09 Thread Xinran Tinney
> On Jan. 9, 2018, 6:44 p.m., Na Li wrote: > > sentry-binding/sentry-binding-hive-common/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzBinding.java > > Line 394 (original), 394 (patched) > > > > > > why

Re: Review Request 65046: SENTRY-1819 HMSFollower and friends do not belong in sentry.service.thrift

2018-01-09 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65046/#review195057 --- sentry-binding/sentry-binding-hive-common/src/main/java/org/apach

Re: Review Request 65046: SENTRY-1819 HMSFollower and friends do not belong in sentry.service.thrift

2018-01-09 Thread Xinran Tinney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65046/ --- (Updated Jan. 9, 2018, 5:32 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 65046: SENTRY-1819 HMSFollower and friends do not belong in sentry.service.thrift

2018-01-09 Thread Steve Moist via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65046/#review195043 --- sentry-dist/src/license/THIRD-PARTY.properties Line 1 (original),

Review Request 65046: SENTRY-1819 HMSFollower and friends do not belong in sentry.service.thrift

2018-01-09 Thread Xinran Tinney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65046/ --- Review request for sentry, Alexander Kolbasov, Arjun Mishra, Colm O hEigeartaigh