Review Request 56481: SENTRY-1625: PrivilegeOperatePersistence can use QueryParamBuilder

2017-02-08 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56481/ --- Review request for sentry, Hao Hao, kalyan kumar kalvagadda, Vamsee Yarlagadda,

Sentry-jdk-1.7-v2 - Build # 63 - Fixed

2017-02-08 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7-v2 (build #63) Status: Fixed Check console output at https://builds.apache.org/job/Sentry-jdk-1.7-v2/63/ to view the results.

Re: Review Request 56454: SENTRY-1609: DelegateSentryStore is subject to JDQL injection

2017-02-08 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56454/#review164760 --- Ship it! Ship It! - kalyan kumar kalvagadda On Feb. 8, 2017,

Re: Review Request 56454: SENTRY-1609: DelegateSentryStore is subject to JDQL injection

2017-02-08 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56454/#review164753 --- Ship it! Ship It! - Vamsee Yarlagadda On Feb. 8, 2017, 5:34

Re: Review Request 56411: SENTRY-1624 DefaultSentryValidator doesn't correctly construct SentryOnFailureHookContextImpl

2017-02-08 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56411/#review164752 --- Ship it! Ship It! - Vamsee Yarlagadda On Feb. 8, 2017, 5:15

Review Request 56458: SENTRY-1614 Removed code having PATHS column as primary key. Primary key of this long is not accepted by all databases

2017-02-08 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56458/ --- Review request for sentry, Alexander Kolbasov, Hao Hao, Vamsee Yarlagadda, and V

Re: Review Request 56403: SENTRY-1387 Add HDFS sync tests for drop partition for external/implicit locations

2017-02-08 Thread Nachiket Vaidya
> On Feb. 8, 2017, 6:59 p.m., Hao Hao wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/metastore/TestSentryListenerSentryDeserializer.java, > > line 232 > > > > > > Extra space? w

Re: Review Request 56403: SENTRY-1387 Add HDFS sync tests for drop partition for external/implicit locations

2017-02-08 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56403/#review164740 --- Fix it, then Ship it! sentry-tests/sentry-tests-hive/src/test/

Re: Review Request 56356: SENTRY-1615 SentryStore should not allocate empty objects that are immediately returned

2017-02-08 Thread Alexander Kolbasov
> On Feb. 8, 2017, 2:35 p.m., kalyan kumar kalvagadda wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/DelegateSentryStore.java, > > line 286 > > > >

Review Request 56454: SENTRY-1609: DelegateSentryStore is subject to JDQL injection

2017-02-08 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56454/ --- Review request for sentry, Hao Hao, kalyan kumar kalvagadda, Vamsee Yarlagadda,

Re: Review Request 56411: SENTRY-1624 DefaultSentryValidator doesn't correctly construct SentryOnFailureHookContextImpl

2017-02-08 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56411/#review164706 --- Ship it! Ship It! - kalyan kumar kalvagadda On Feb. 8, 2017,

Re: Review Request 56411: SENTRY-1624 DefaultSentryValidator doesn't correctly construct SentryOnFailureHookContextImpl

2017-02-08 Thread Nachiket Vaidya
> On Feb. 8, 2017, 2:19 p.m., kalyan kumar kalvagadda wrote: > > sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryValidator.java, > > line 213 > > > > > >

Re: Review Request 56403: SENTRY-1387 Add HDFS sync tests for drop partition for external/implicit locations

2017-02-08 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56403/#review164700 --- Ship it! Ship It! - kalyan kumar kalvagadda On Feb. 7, 2017,

Re: Review Request 56356: SENTRY-1615 SentryStore should not allocate empty objects that are immediately returned

2017-02-08 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56356/#review164562 --- sentry-service/sentry-service-server/src/main/java/org/apache/sen

Re: Review Request 56411: SENTRY-1624 DefaultSentryValidator doesn't correctly construct SentryOnFailureHookContextImpl

2017-02-08 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56411/#review164684 --- sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/se

FINAL REMINDER: CFP for ApacheCon closes February 11th

2017-02-08 Thread Rich Bowen
Dear Apache Enthusiast, This is your FINAL reminder that the Call for Papers (CFP) for ApacheCon Miami is closing this weekend - February 11th. This is your final opportunity to submit a talk for consideration at this event. This year, we are running several mini conferences in conjunction with t

Sentry-jdk-1.7-v2 - Build # 62 - Unstable

2017-02-08 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7-v2 (build #62) Status: Unstable Check console output at https://builds.apache.org/job/Sentry-jdk-1.7-v2/62/ to view the results.

Sentry-jdk-1.7 - Build # 761 - Failure

2017-02-08 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7 (build #761) Status: Failure Check console output at https://builds.apache.org/job/Sentry-jdk-1.7/761/ to view the results.