Re: Review Request 55706: SENTRY-1566: Make full Perm/Path snapshot available for NN plugin

2017-03-07 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55706/#review168248 --- My comments regarding the use of interfaces (List, Map) rather the

Re: Review Request 55706: SENTRY-1566: Make full Perm/Path snapshot available for NN plugin

2017-03-07 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55706/#review168247 --- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hd

Re: Review Request 55706: SENTRY-1566: Make full Perm/Path snapshot available for NN plugin

2017-03-07 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55706/#review168230 --- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hd

Re: Review Request 56954: SENTRY-1639 Refactor the usage of configuration constants related to transport

2017-03-07 Thread kalyan kumar kalvagadda
> On March 7, 2017, 1:17 a.m., Alexander Kolbasov wrote: > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryClientTransportConfigInterface.java > > Lines 27 (patched) > > > > >

Re: Review Request 56954: SENTRY-1639 Refactor the usage of configuration constants related to transport

2017-03-07 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56954/ --- (Updated March 7, 2017, 6:51 p.m.) Review request for sentry, Alexander Kolbaso

Review Request 57375: Unable to truncate table .; from "default" databases

2017-03-07 Thread Yongzhi Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57375/ --- Review request for sentry and Hao Hao. Repository: sentry Description ---