Repository: hadoop Updated Branches: refs/heads/trunk 2326171ea -> 4492b9e73
HADOOP-12294. Remove the support of the deprecated dfs.umask. Contributed by Chang Li. Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/4492b9e7 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/4492b9e7 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/4492b9e7 Branch: refs/heads/trunk Commit: 4492b9e7302b8c84dddec9713d8148cc6183f46b Parents: 2326171 Author: Haohui Mai <[email protected]> Authored: Sun Nov 22 16:47:44 2015 -0800 Committer: Haohui Mai <[email protected]> Committed: Sun Nov 22 16:47:44 2015 -0800 ---------------------------------------------------------------------- hadoop-common-project/hadoop-common/CHANGES.txt | 3 +++ .../apache/hadoop/fs/permission/FsPermission.java | 18 ++---------------- 2 files changed, 5 insertions(+), 16 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/4492b9e7/hadoop-common-project/hadoop-common/CHANGES.txt ---------------------------------------------------------------------- diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index 52dea6e..ebde7b7 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -35,6 +35,9 @@ Trunk (Unreleased) HADOOP-10787 Rename/remove non-HADOOP_*, etc from the shell scripts. (aw via vvasudev) + HADOOP-12294. Remove the support of the deprecated dfs.umask. + (Chang Li vha wheat9) + NEW FEATURES HADOOP-6590. Add a username check for hadoop sub-commands (John Smith via http://git-wip-us.apache.org/repos/asf/hadoop/blob/4492b9e7/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/FsPermission.java ---------------------------------------------------------------------- diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/FsPermission.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/FsPermission.java index 0258293..d4adbb5 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/FsPermission.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/FsPermission.java @@ -258,22 +258,8 @@ public class FsPermission implements Writable { String error = "Unable to parse configuration " + UMASK_LABEL + " with value " + confUmask + " as " + type + " umask."; LOG.warn(error); - - // If oldUmask is not set, then throw the exception - if (oldUmask == Integer.MIN_VALUE) { - throw new IllegalArgumentException(error); - } - } - - if(oldUmask != Integer.MIN_VALUE) { // Property was set with old key - if (umask != oldUmask) { - LOG.warn(DEPRECATED_UMASK_LABEL - + " configuration key is deprecated. " + "Convert to " - + UMASK_LABEL + ", using octal or symbolic umask " - + "specifications."); - // Old and new umask values do not match - Use old umask - umask = oldUmask; - } + + throw new IllegalArgumentException(error); } }
