Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-08-02 Thread Sravya Tirukkovalur
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/#review144572 --- Ship it! Ship It! - Sravya Tirukkovalur On Aug. 2, 2016, 11:

Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-08-02 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/ --- (Updated Aug. 2, 2016, 11:27 p.m.) Review request for sentry, Anne Yu and Sravy

Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-08-02 Thread Hao Hao
> On July 29, 2016, 8:03 p.m., Sravya Tirukkovalur wrote: > > Seems like we need to not block Sentry startup for the sake of HMS full > > update (to avoid dependencies at startup), but would like to not serve > > requests ( eventually only for Hive requests) until the full update > > processin

Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-08-02 Thread Hao Hao
> On July 28, 2016, 6:08 p.m., Rahul Sharma wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java, > > line 169 > > > > > > Handle exector shutdown in case o

Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-07-29 Thread Sravya Tirukkovalur
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/#review144165 --- Seems like we need to not block Sentry startup for the sake of HMS

Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-07-29 Thread Sravya Tirukkovalur
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/#review144152 --- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hd

Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-07-28 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/ --- (Updated July 29, 2016, 1:34 a.m.) Review request for sentry, Anne Yu and Sravy

Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-07-28 Thread Rahul Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/#review143970 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-07-20 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/ --- (Updated July 21, 2016, 6:22 a.m.) Review request for sentry, Anne Yu and Sravy

Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-07-20 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/ --- (Updated July 21, 2016, 6:11 a.m.) Review request for sentry, Anne Yu and Sravy

Re: Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-07-20 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/ --- (Updated July 21, 2016, 6:01 a.m.) Review request for sentry, Anne Yu and Sravy

Review Request 50182: Sentry-1371: Rework Sentry start up and Hive state fetch

2016-07-19 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50182/ --- Review request for sentry, Anne Yu and Sravya Tirukkovalur. Repository: sentry