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!

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

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

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