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

2017-12-13 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64545/#review193753 --- Ship it! Ship It! - Alexander Kolbasov On Dec. 14, 2017,

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

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

Re: Review Request 64471: SENTRY-2096: Fail unit tests at end

2017-12-13 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64471/ --- (Updated Dec. 13, 2017, 10:29 p.m.) Review request for sentry, kalyan kumar

Re: Issue with SimpleCacheProviderBackend

2017-12-13 Thread Na Li
Sasha, sentry-1291 is helpful for the problem that sentry privilege checks takes too long with many explicit grants, which is useful for big customers. Another approach that can improve the performance is to organize the privileges according to the authorization hierarchy in a tree structure, so

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

2017-12-13 Thread Arjun Mishra via Review Board
> On Dec. 13, 2017, 6:42 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/LeaderStatusMonitor.java > > Line 281 (original), 282 (patched) > > > > >

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

2017-12-13 Thread Alexander Kolbasov
> On Dec. 13, 2017, 6:42 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/LeaderStatusMonitor.java > > Line 281 (original), 282 (patched) > > > > >

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

2017-12-13 Thread Arjun Mishra via Review Board
> On Dec. 13, 2017, 6:42 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/LeaderStatusMonitor.java > > Line 281 (original), 282 (patched) > > > > >

Re: 1.7.1 version in JIRA

2017-12-13 Thread Alexander Kolbasov
Done On Wed, Dec 13, 2017 at 6:26 AM, Colm O hEigeartaigh wrote: > Can someone add a 1.7.1 version in JIRA please? > > Colm. > > > -- > Colm O hEigeartaigh > > Talend Community Coder > http://coders.talend.com >

Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2017-12-13 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/ --- Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, Sergio Pena,

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

2017-12-13 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64545/#review193716 ---

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

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

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

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

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

2017-12-13 Thread Arjun Mishra via Review Board
> On Dec. 12, 2017, 7:01 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/LeaderStatusMonitor.java > > Line 254 (original), 255 (patched) > > > > > >