Re: Review Request 66675: SENTRY-2203: Leader Lock is not released when Sentry service shuts down

2018-04-17 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66675/ --- (Updated April 17, 2018, 11:44 p.m.) Review request for sentry, Alexander Kolba

Re: Review Request 66675: SENTRY-2203: Leader Lock is not released when Sentry service shuts down

2018-04-17 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66675/ --- (Updated April 17, 2018, 11:15 p.m.) Review request for sentry, Alexander Kolba

Re: Review Request 66675: SENTRY-2203: Leader Lock is not released when Sentry service shuts down

2018-04-17 Thread Na Li via Review Board
> On April 17, 2018, 10:38 p.m., Sergio Pena wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/HAContext.java > > Line 200 (original), 200-202 (patched) > > > >

Re: Review Request 66675: SENTRY-2203: Leader Lock is not released when Sentry service shuts down

2018-04-17 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66675/#review201354 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Review Request 66675: SENTRY-2203: Leader Lock is not released when Sentry service shuts down

2018-04-17 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66675/ --- Review request for sentry, Alexander Kolbasov, Arjun Mishra, and Sergio Pena. R

Sentry-jdk-1.8 - Build # 14 - Failure

2018-04-17 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.8 (build #14) Status: Failure Check console output at https://builds.apache.org/job/Sentry-jdk-1.8/14/ to view the results.

Re: Review Request 66636: SENTRY-2200 Update Sentry datanucleus config names

2018-04-17 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66636/#review201324 --- Ship it! Ship It! - Na Li On April 16, 2018, 4:31 p.m., Xinr

Re: Review Request 66636: SENTRY-2200 Update Sentry datanucleus config names

2018-04-17 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66636/#review201315 --- Ship it! It looks good, thanks for the patch. - Sergio Pena

Re: Review Request 66337: SENTRY-2167: Fix the logs in NotificationProcessor

2018-04-17 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66337/#review201314 --- Ship it! Ship It! - Sergio Pena On March 28, 2018, 4:16 p.m.