Re: Review Request 63596: SENTRY-1951 - Old SentryStore.retrieveFullPathsImage() should be removed

2017-12-11 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63596/#review193478 --- Ship it! Ship It! sentry-hdfs/sentry-hdfs-service/src/test/ja

Re: Review Request 63596: SENTRY-1951 - Old SentryStore.retrieveFullPathsImage() should be removed

2017-12-11 Thread Arjun Mishra via Review Board
> On Nov. 30, 2017, 3:09 p.m., Xinran Tinney wrote: > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java > > Lines 3574-3580 (patched) > > > > > >

Re: Update to the Wiki page on How to release sentry(Review)

2017-12-11 Thread Sergio Pena
It looks good to me Kalyan. On Mon, Dec 11, 2017 at 8:46 AM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > Hello all, > > Based on my experience in releasing sentry 2.0.0, I have created new page > documenting the steps. I have used the older page as a base line and > updated stuff. >

Re: [ANNOUNCE] Apache Sentry 2.0.0 released

2017-12-11 Thread Sergio Pena
Hey An, Upgrading Hive on Spark is more a question for the Spark community. They forked Hive 1.2 and created their own hive-spark repo due to some incompatibilities with Hive 2.x. In regards to Sentry, it will be hard to support Spark 2.2.1 which bundles Hive 1.2 because Sentry 2.0 requires the u

Re: Review Request 63596: SENTRY-1951 - Old SentryStore.retrieveFullPathsImage() should be removed

2017-12-11 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63596/ --- (Updated Dec. 11, 2017, 9:02 p.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 64259: SENTRY-1572 SentryMain() shouldn't dynamically load tool class

2017-12-11 Thread Xinran Tinney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64259/ --- (Updated Dec. 11, 2017, 8:03 p.m.) Review request for sentry, Alexander Kolbaso

Update to the Wiki page on How to release sentry(Review)

2017-12-11 Thread Kalyan Kumar Kalvagadda
Hello all, Based on my experience in releasing sentry 2.0.0, I have created new page documenting the steps. I have used the older page as a base line and updated stuff. I have created new page just for review. After handling review comments, I will update the current page that way the page histor