Re: Review Request 58166: SENTRY-1692 ZK namespace configuration doesn't work

2017-04-06 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58166/#review171310 --- Ship it! Ship It! - Hao Hao On April 4, 2017, 5:03 a.m.,

Re: Review Request 58166: SENTRY-1692 ZK namespace configuration doesn't work

2017-04-04 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58166/#review171080 --- Ship it! Ship It! - Na Li On April 4, 2017, 5:03 a.m.,

Re: Review Request 58166: SENTRY-1692 ZK namespace configuration doesn't work

2017-04-04 Thread Alexander Kolbasov
> On April 4, 2017, 6:07 a.m., Hao Hao wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/LeaderStatusMonitor.java > > Line 96 (original) > > > > > > Not quite follow the

Re: Review Request 58166: SENTRY-1692 ZK namespace configuration doesn't work

2017-04-04 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58166/#review170957 ---