Re: Review Request 46077: SENTRY-1184: Clean up HMSPaths.renameAuthzObject

2016-04-12 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46077/#review128579 --- Fix it, then Ship it! Ship It!

Re: Review Request 46077: SENTRY-1184: Clean up HMSPaths.renameAuthzObject

2016-04-12 Thread Hao Hao
> On April 12, 2016, 11:22 p.m., Sravya Tirukkovalur wrote: > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java, > > line 567 > > > > > > By managed table I mean, when table rename

Re: Review Request 46077: SENTRY-1184: Clean up HMSPaths.renameAuthzObject

2016-04-12 Thread Hao Hao
> On April 12, 2016, 11:22 p.m., Sravya Tirukkovalur wrote: > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java, > > line 263 > > > > > > Log is actually added here. Am I missing

Re: Review Request 46077: SENTRY-1184: Clean up HMSPaths.renameAuthzObject

2016-04-12 Thread Sravya Tirukkovalur
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46077/#review128527 ---

Re: Review Request 46077: SENTRY-1184: Clean up HMSPaths.renameAuthzObject

2016-04-12 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46077/#review128507 ---