Sentry-jdk-1.7-v2 - Build # 124 - Still Failing

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

Re: Review Request 61793: SENTRY-1894: Update field size in package.jdo for dataNucleus to match size in sql

2017-08-28 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61793/#review183954 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 61793: SENTRY-1894: Update field size in package.jdo for dataNucleus to match size in sql

2017-08-28 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61793/#review183955 --- For MySQL I see this: ALTER TABLE `SENTRY_DB_PRIVILEGE` ADD

Re: Review Request 61793: SENTRY-1894: Update field size in package.jdo for dataNucleus to match size in sql

2017-08-28 Thread Na Li
> On Aug. 28, 2017, 5:03 p.m., Vamsee Yarlagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > > Line 135 (original), 126 (patched) > > > > > >

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-08-28 Thread Sergio Pena
Kalyan, I cherry-picked HIVE-17048 to all 2.x versions: 2.1.2, 2.2.1,2.3.1 and 2.4.0 They are not released yet, but as soon as one of them is released, then we can bump the Hive version of Sentry to the new version. - Sergio On Fri, Aug 11, 2017 at 12:30 PM, Kalyan Kumar Kalvagadda < kkal...@cl

Re: Review Request 61793: SENTRY-1894: Update field size in package.jdo for dataNucleus to match size in sql

2017-08-28 Thread Na Li
> On Aug. 28, 2017, 5:16 p.m., Alexander Kolbasov wrote: > > For MySQL I see this: > > > > ALTER TABLE `SENTRY_DB_PRIVILEGE` > > ADD UNIQUE `SENTRY_DB_PRIV_PRIV_NAME_UNIQ` > > (`SERVER_NAME`,`DB_NAME`,`TABLE_NAME`,`COLUMN_NAME`,`URI`(250),`ACTION`,`WITH_GRANT_OPTION`); > > > > Does it ma

Re: Review Request 61793: SENTRY-1894: Update field size in package.jdo for dataNucleus to match size in sql

2017-08-28 Thread Alexander Kolbasov
> On Aug. 28, 2017, 5:16 p.m., Alexander Kolbasov wrote: > > For MySQL I see this: > > > > ALTER TABLE `SENTRY_DB_PRIVILEGE` > > ADD UNIQUE `SENTRY_DB_PRIV_PRIV_NAME_UNIQ` > > (`SERVER_NAME`,`DB_NAME`,`TABLE_NAME`,`COLUMN_NAME`,`URI`(250),`ACTION`,`WITH_GRANT_OPTION`); > > > > Does it ma

Re: Review Request 61793: SENTRY-1894: Update field size in package.jdo for dataNucleus to match size in sql

2017-08-28 Thread Na Li
> On Aug. 28, 2017, 5:16 p.m., Alexander Kolbasov wrote: > > For MySQL I see this: > > > > ALTER TABLE `SENTRY_DB_PRIVILEGE` > > ADD UNIQUE `SENTRY_DB_PRIV_PRIV_NAME_UNIQ` > > (`SERVER_NAME`,`DB_NAME`,`TABLE_NAME`,`COLUMN_NAME`,`URI`(250),`ACTION`,`WITH_GRANT_OPTION`); > > > > Does it ma

Re: Review Request 61889: SENTRY-1895: Sentry should handle the case of multiple notifications with the same ID

2017-08-28 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61889/#review183881 --- Fix it, then Ship it! Ship It! sentry-provider/sentry-provide

Re: Review Request 61793: SENTRY-1894: Update field size in package.jdo for dataNucleus to match size in sql

2017-08-28 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61793/ --- (Updated Aug. 28, 2017, 11:09 p.m.) Review request for sentry, Alexander Kolbas