Re: Review Request 64179: SENTRY-2079: Sentry HA leader monitor does not work due to a mix of curator versions in the classpath

2017-11-30 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64179/#review192434 --- Ship it! Ship It! - kalyan kumar kalvagadda On Nov. 29, 2017

Re: Review Request 64179: SENTRY-2079: Sentry HA leader monitor does not work due to a mix of curator versions in the classpath

2017-11-29 Thread Sergio Pena via Review Board
> On Nov. 30, 2017, 2:58 a.m., Na Li wrote: > > sentry-provider/sentry-provider-db/pom.xml > > Lines 230 (patched) > > > > > > do you need to add this in both root pom.xml line 635 and here? It's not necessary. Tihs

Re: Review Request 64179: SENTRY-2079: Sentry HA leader monitor does not work due to a mix of curator versions in the classpath

2017-11-29 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64179/#review192231 --- Ship it! Ship It! - Na Li On Nov. 29, 2017, 10:37 p.m., Serg

Re: Review Request 64179: SENTRY-2079: Sentry HA leader monitor does not work due to a mix of curator versions in the classpath

2017-11-29 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64179/#review192228 --- sentry-provider/sentry-provider-db/pom.xml Lines 230 (patched)

Review Request 64179: SENTRY-2079: Sentry HA leader monitor does not work due to a mix of curator versions in the classpath

2017-11-29 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64179/ --- Review request for sentry. Bugs: sentry-2079 https://issues.apache.org/jira