Re: Review Request 59424: SENTRY-1736 Generic service client should support Kerberos(Continuation Fix)

2017-05-19 Thread Vamsee Yarlagadda
> On May 20, 2017, 12:23 a.m., Vamsee Yarlagadda wrote: > > Ship It! Yeah update the review with the new JIRA number. - Vamsee --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59424/#review175

Re: Review Request 59424: SENTRY-1736 Generic service client should support Kerberos(Continuation Fix)

2017-05-19 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59424/#review175585 --- Ship it! Ship It! - Vamsee Yarlagadda On May 19, 2017, 11:27

Re: Review Request 59424: SENTRY-1736 Generic service client should support Kerberos(Continuation Fix)

2017-05-19 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59424/#review175584 --- The SENTRY-1736 is already comitted and resolved. Please file anot

Review Request 59424: SENTRY-1736 Generic service client should support Kerberos(Continuation Fix)

2017-05-19 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59424/ --- Review request for sentry, Alexander Kolbasov and Vamsee Yarlagadda. Bugs: SENT

Re: Review Request 59414: SENTRY-1771: HDFS client requests full permission update multiple times.

2017-05-19 Thread Alexander Kolbasov
> On May 19, 2017, 8:22 p.m., Sergio Pena wrote: > > sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java > > Lines 217 (patched) > > > > > > If the thread is sche

Re: Review Request 59414: SENTRY-1771: HDFS client requests full permission update multiple times.

2017-05-19 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59414/#review175567 --- sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-19 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/#review175558 --- sentry-provider/sentry-provider-db/src/test/java/org/apache/sentr

Review Request 59414: SENTRY-1771: HDFS client requests full permission update multiple times.

2017-05-19 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59414/ --- Review request for sentry, Alexander Kolbasov, Hao Hao, kalyan kumar kalvagadda,

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-19 Thread Vadim Spector
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/#review175553 --- Ship it! Ship It! - Vadim Spector On May 19, 2017, 1:18 a.m.

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-19 Thread Vadim Spector
> On May 19, 2017, 6:26 p.m., Vadim Spector wrote: > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/UserGroupInformationInitializer.java > > Line 36 (original), 40 (patched) > > >

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-19 Thread Vamsee Yarlagadda
> On May 19, 2017, 6:26 p.m., Vadim Spector wrote: > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/UserGroupInformationInitializer.java > > Line 36 (original), 40 (patched) > > >

Re: Review Request 59317: SENTRY-1736 Generic service client should support Kerberos

2017-05-19 Thread Vadim Spector
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59317/#review175543 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/co

Re: Review Request 59397: SENTRY-1776 SentryStore should clear SENTRY_HMS_NOTIFICATION_ID while clearing store

2017-05-19 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59397/#review175506 --- Ship it! Ship It! - Alexander Kolbasov On May 19, 2017, 1:17

Re: Review Request 59397: SENTRY-1776 SentryStore should clear SENTRY_HMS_NOTIFICATION_ID while clearing store

2017-05-19 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59397/#review175501 --- Ship it! Ship It! - Sergio Pena On May 19, 2017, 1:17 p.m.,

Review Request 59397: SENTRY-1776 SentryStore should clear SENTRY_HMS_NOTIFICATION_ID while clearing store

2017-05-19 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59397/ --- Review request for sentry, Alexander Kolbasov, Brian Towles, Hao Hao, Na Li, Ser