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

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:

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

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 ---

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:

Review Request 58069: SENTRY-1670 Expose current HMS notification ID as a Sentry gauge (metric)

2017-03-30 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58069/ --- Review request for sentry. Bugs: SENTRY-1670