Mailing-List: contact hdfs-issues-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: hdfs-issues@hadoop.apache.org
Date: Sat, 27 Sep 2014 00:55:35 +0000 (UTC)
From: "Jing Zhao (JIRA)" <jira@apache.org>
To: hdfs-issues@hadoop.apache.org
Message-ID: <JIRA.12742634.1411092786000.136743.1411779335622@Atlassian.JIRA>
In-Reply-To: <JIRA.12742634.1411092786000@Atlassian.JIRA>
References: <JIRA.12742634.1411092786000@Atlassian.JIRA>
 <JIRA.12742634.1411092786539@arcas>
Subject: [jira] [Commented] (HDFS-7093) Allow unprivileged users to set
 storage policy
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/HDFS-7093?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14150290#comment-14150290 ] 

Jing Zhao commented on HDFS-7093:
---------------------------------

Only one nit: for the following code maybe we can also check if permission is enabled before initializing pc. Other than this +1.
{code}
+    FSPermissionChecker pc = null;
+    if (restrictSetStoragePolicy) {
+      checkSuperuserPrivilege();
+    } else {
+      pc = getPermissionChecker();
+    }
{code}


> Allow unprivileged users to set storage policy
> ----------------------------------------------
>
>                 Key: HDFS-7093
>                 URL: https://issues.apache.org/jira/browse/HDFS-7093
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: hdfs-client, namenode
>    Affects Versions: 3.0.0
>            Reporter: Arpit Agarwal
>            Assignee: Arpit Agarwal
>         Attachments: HDFS-7093.01.patch, HDFS-7093.02.patch, HDFS-7093.03.patch
>
>
> Currently we do not allow unprivileged users to set the storage policy on a directory. This Jira proposes removing this restriction.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)