Issue with SimpleCacheProviderBackend

2017-12-12 Thread Colm O hEigeartaigh
Hi all, I've updated some local testcases to work with Sentry 2.0.0 and the "v1" Hive binding (previously working fine using 1.8.0 and the "v2" binding). I have a simple table called "words" (word STRING, count INT). I am making an SQL call as the user "bob", e.g. "SELECT * FROM words where count

Review Request 64545: SENTRY-2078: Have sentry server print an obvious INFO level log message when it becomes the writer

2017-12-12 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64545/ --- Review request for sentry. Repository: sentry Description --- Currently

Re: Review Request 64545: SENTRY-2078: Have sentry server print an obvious INFO level log message when it becomes the writer

2017-12-12 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64545/ --- (Updated Dec. 12, 2017, 4:08 p.m.) Review request for sentry, Brian Towles, kal

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

2017-12-12 Thread Kalyan Kumar Kalvagadda
I will wait another day before making this page official. -Kalyan On Mon, Dec 11, 2017 at 5:21 PM, Sergio Pena wrote: > 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 re

Re: Issue with SimpleCacheProviderBackend

2017-12-12 Thread Na Li
Colm, I suspect it is a bug in SENTRY-1291. I can take a look later today. Thanks, Lina On Tue, Dec 12, 2017 at 4:32 AM, Colm O hEigeartaigh wrote: > Hi all, > > I've updated some local testcases to work with Sentry 2.0.0 and the "v1" > Hive binding (previously working fine using 1.8.0 and th

Re: Review Request 64545: SENTRY-2078: Have sentry server print an obvious INFO level log message when it becomes the writer

2017-12-12 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64545/#review193563 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 64545: SENTRY-2078: Have sentry server print an obvious INFO level log message when it becomes the writer

2017-12-12 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64545/ --- (Updated Dec. 12, 2017, 5:10 p.m.) Review request for sentry, Brian Towles, kal

Re: Issue with SimpleCacheProviderBackend

2017-12-12 Thread Colm O hEigeartaigh
That would be great, thanks! Colm. On Tue, Dec 12, 2017 at 4:36 PM, Na Li wrote: > Colm, > > I suspect it is a bug in SENTRY-1291. I can take a look later today. > > Thanks, > > Lina > > On Tue, Dec 12, 2017 at 4:32 AM, Colm O hEigeartaigh > wrote: > > > Hi all, > > > > I've updated some local

Re: Review Request 64545: SENTRY-2078: Have sentry server print an obvious INFO level log message when it becomes the writer

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

Re: Review Request 64545: SENTRY-2078: Have sentry server print an obvious INFO level log message when it becomes the writer

2017-12-12 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64545/#review193591 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

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

2017-12-12 Thread Arjun Mishra via Review Board
> On Dec. 12, 2017, 1:26 a.m., Alexander Kolbasov wrote: > > sentry-hdfs/sentry-hdfs-service/src/test/java/org/apache/sentry/hdfs/TestImageRetriever.java > > Line 57 (original), 56 (patched) > > > > > > Not necessaril

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

2017-12-12 Thread Alexander Kolbasov
> On Dec. 12, 2017, 1:26 a.m., Alexander Kolbasov wrote: > > sentry-hdfs/sentry-hdfs-service/src/test/java/org/apache/sentry/hdfs/TestImageRetriever.java > > Line 57 (original), 56 (patched) > > > > > > Not necessaril

Re: Issue with SimpleCacheProviderBackend

2017-12-12 Thread Na Li
Colm, I did not get chance to look into this issue today. Sorry about that. You can add a e2e test case and set break point at where the authorization object hierarchy to a list of authorization objects, which is used to do exact match with cache Sent from my iPhone > On Dec 12, 2017, at 11:2