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

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

Re: Review Request 58412: [WIP]SENTRY-1587: Refactor SentryStore transaction to persist a single path transcation bundled with corresponding delta path change

2017-04-14 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58412/#review172041 --- - kalyan kumar kalvagadda On April 13, 2017, 6:58 a.m., Hao Hao

Re: Review Request 58412: [WIP]SENTRY-1587: Refactor SentryStore transaction to persist a single path transcation bundled with corresponding delta path change

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

Re: Review Request 58221: SENTRY-1649 move HMS follower to runServer

2017-04-14 Thread Na Li
> On April 14, 2017, 7:04 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java > > Lines 221 (patched) > > > > > > SentryStore

Re: Review Request 58221: SENTRY-1649 move HMS follower to runServer

2017-04-14 Thread Na Li
> On April 14, 2017, 12:57 a.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java > > Lines 368 (patched) > > > > > > 10 seconds seems

Re: Review Request 58221: SENTRY-1649 move HMS follower to runServer

2017-04-14 Thread Na Li
> On April 12, 2017, 4:50 a.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java > > Lines 78 (patched) > > > > > > Do you ever expect

Re: Review Request 58221: SENTRY-1649 move HMS follower to runServer

2017-04-14 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58221/ --- (Updated April 14, 2017, 8:36 p.m.) Review request for sentry, Alexander Kolbas

Re: Review Request 58221: SENTRY-1649 move HMS follower to runServer

2017-04-14 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58221/#review172014 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 58221: SENTRY-1649 move HMS follower to runServer

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

Re: Review Request 58221: SENTRY-1649 move HMS follower to runServer

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

Re: Review Request 58412: [WIP]SENTRY-1587: Refactor SentryStore transaction to persist a single path transcation bundled with corresponding delta path change

2017-04-14 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58412/#review172007 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 58221: SENTRY-1649 move HMS follower to runServer

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

Re: Review Request 58221: SENTRY-1649 move HMS follower to runServer

2017-04-14 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58221/ --- (Updated April 14, 2017, 3:41 p.m.) Review request for sentry, Alexander Kolbas