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

2017-01-31 Thread Vadim Spector
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56134/#review163726 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/Se

Re: Review Request 55999: SENTRY-1602 Code cleanup for Sentry JSON message factory for hive notifications

2017-01-31 Thread Nachiket Vaidya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55999/ --- (Updated Jan. 31, 2017, 11:27 p.m.) Review request for sentry, Alexander Kolbas

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

2017-01-31 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56134/ --- (Updated Jan. 31, 2017, 8:39 p.m.) Review request for sentry, Alexander Kolbaso

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

2017-01-31 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56134/ --- Review request for sentry, Alexander Kolbasov, Hao Hao, Mat Crocker, Vamsee Yarl

Re: Review Request 55246: SENTRY-1536: Refactor SentryStore transaction management to allow for extra transactions for a single permission update

2017-01-31 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55246/#review163654 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr