----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12049/ -----------------------------------------------------------
(Updated June 22, 2013, 6:35 p.m.) Review request for hive. Changes ------- Update the diff Description ------- The changes include: 1. a new property hive.scratch.dir.permission with default value 700 to control the permission of dirs created directly under scratchdir (local and hdfs) 2. only the directories directly under hdfs /tmp/hive-{username} or local /tmp/{username} will be set to the permission as specified by above property. These include ones created for the mapred jobs. By doing so, we will add the protection to newly created dirs/files with minimal code change, and also preserve the permissions of the existing scratchdirs (for back-compatibility). This addresses bug HIVE-4487. https://issues.apache.org/jira/browse/HIVE-4487 Diffs (updated) ----- common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 73ec6f2 ql/src/java/org/apache/hadoop/hive/ql/Context.java 5340e99 Diff: https://reviews.apache.org/r/12049/diff/ Testing ------- Manual tests Thanks, Chaoyu Tang