Sentry-jdk-1.7-v2 - Build # 98 - Still unstable

2017-07-19 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7-v2 (build #98) Status: Still unstable Check console output at https://builds.apache.org/job/Sentry-jdk-1.7-v2/98/ to view the results.

Re: Review Request 60926: SENTRY-1760: HMSFollower should detect when a full snapshot from HMS is required

2017-07-19 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60926/#review180946 ---

Re: Review Request 60904: SENTRY-1849 Fix the pom file to use appropriate hive dependencies based on hive-authz profile

2017-07-19 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60904/#review180910 --- Ship it! Ship It! - Colm O hEigeartaigh On July 18, 2017,

Re: Review Request 60869: SENTRY-1846 - Use a consistent configuration variable for the sentry provider property

2017-07-19 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60869/ --- (Updated July 19, 2017, 9:16 a.m.) Review request for sentry. Bugs:

Re: Review Request 60926: SENTRY-1760: HMSFollower should detect when a full snapshot from HMS is required

2017-07-19 Thread Sergio Pena
> On July 19, 2017, 9:27 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Lines 169 (patched) > > > > > > Currently,

Re: Review Request 60926: SENTRY-1760: HMSFollower should detect when a full snapshot from HMS is required

2017-07-19 Thread Sergio Pena
> On July 19, 2017, 9:27 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Lines 193-214 (patched) > > > > > > With your

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-07-19 Thread Kalyan Kumar Kalvagadda
Hello all, Either way we need to fix couple of things first. 1. Fix the test failures when hive-authz2 and datanucleus4 profiles are enabled. 2. Update e2e tests in sentry-tests-hive-v2 package to have same coverage as tests in sentry-tests-hive package. Does any one wants to take ownership of

Re: Review Request 60869: SENTRY-1846 - Use a consistent configuration variable for the sentry provider property

2017-07-19 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60869/#review180929 --- Ship it! Ship It! - Na Li On July 19, 2017, 9:16 a.m., Colm