Re: Review Request 57901: SENTRY-1593 Implement client failover for all the sentry clients

2017-03-30 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57901/#review170671 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/co

Re: Review Request 56134: SENTRY-1593 Implementing client failover for Generic policy clients and namenode clients

2017-03-30 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56134/#review170667 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/co

Review Request 58094: SENTRY-1683 MetastoreCacheInitializer has a race condition in handling results list

2017-03-30 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58094/ --- Review request for sentry, Hao Hao, kalyan kumar kalvagadda, Na Li, and Vadim Sp

Review Request 58093: SENTRY-1676: FullUpdateInitializer#createInitialUpdate should not throw RuntimeExceptio

2017-03-30 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58093/ --- Review request for sentry, Hao Hao, kalyan kumar kalvagadda, and Na Li. Bugs: S

Re: Review Request 57901: SENTRY-1593 Implement client failover for all the sentry clients

2017-03-30 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57901/#review170070 --- I think that this should be split into two parts: 1) Move files a

Re: Review Request 57901: SENTRY-1593 Implement client failover for all the sentry clients

2017-03-30 Thread Lei Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57901/#review170644 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/co

Re: Review Request 58087: SENTRY-1676 FullUpdateInitializer#createInitialUpdate should not throw RuntimeException

2017-03-30 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58087/ --- (Updated March 30, 2017, 6:55 p.m.) Review request for sentry, Hao Hao, kalyan

Review Request 58087: SENTRY-1676 FullUpdateInitializer#createInitialUpdate should not throw RuntimeException

2017-03-30 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58087/ --- Review request for sentry, Hao Hao, kalyan kumar kalvagadda, and Na Li. Bugs: S