-----------------------------------------------------------
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

Reply via email to