Review Request 54409: SENTRY-1476: SentryStore is subject to JDQL injection

2016-12-05 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54409/ --- Review request for sentry, Colin Ma, Hao Hao, kalyan kumar kalvagadda, Sravya Ti

Re: Review Request 54338: SENTRY-1515: Cleanup exception handling in SentryStore

2016-12-05 Thread Vadim Spector
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54338/#review158055 --- sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/h

Re: Review Request 54338: SENTRY-1515: Cleanup exception handling in SentryStore

2016-12-05 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54338/#review158054 --- sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/h

Re: Review Request 53872: SENTRY-1525: Provide script to run Sentry directly from the repo

2016-12-05 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53872/#review158052 --- Can we rename the file to say "run_local_sentry" to explictly stat

Re: Review Request 53872: SENTRY-1525: Provide script to run Sentry directly from the repo

2016-12-05 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53872/#review158051 --- Ship it! Ship It! - Vamsee Yarlagadda On Dec. 3, 2016, 5 a.m