----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20435/ -----------------------------------------------------------
(Updated April 18, 2014, 6:33 p.m.) Review request for hive and Brock Noland. Changes ------- Addressing some review comments. Bugs: HIVE-6916 https://issues.apache.org/jira/browse/HIVE-6916 Repository: hive-git Description ------- This fixes the CopyTask (used by export/import) to also do the permission inheritance semantics, if the flag is on. Like elsewhere in the code, this is using the HDFS shell because it allows specification of recursion (whereas I could not find it in the HDFS-API). Diffs (updated) ----- common/src/java/org/apache/hadoop/hive/common/FileUtils.java b36a016 itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/security/TestFolderPermissions.java a635bb0 ql/src/java/org/apache/hadoop/hive/ql/exec/CopyTask.java b429a58 Diff: https://reviews.apache.org/r/20435/diff/ Testing ------- Added a unit test for the same. Thanks, Szehon Ho