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

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

Re: Review Request 57232: SENTRY-1613: Add propagating logic for Perm/Path updates in Sentry service

2017-03-10 Thread Hao Hao
> On March 6, 2017, 10:38 p.m., Lei Xu wrote: > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/DeltaRetriever.java > > Lines 35 (patched) > > > > > > If we retrieve `Perm` and `Path` separately,

Re: Review Request 57232: SENTRY-1613: Add propagating logic for Perm/Path updates in Sentry service

2017-03-10 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57232/ --- (Updated March 10, 2017, 10:29 p.m.) Review request for sentry. Repository: s

Re: Review Request 57232: SENTRY-1613: Add propagating logic for Perm/Path updates in Sentry service

2017-03-10 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57232/ --- (Updated March 10, 2017, 11:14 p.m.) Review request for sentry. Repository: s

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

2017-03-10 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56954/#review168685 --- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/co