hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "James Clampffer (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-10524) libhdfs++: Implement chmod and chown
Date Wed, 15 Jun 2016 16:39:09 GMT

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

James Clampffer commented on HDFS-10524:
----------------------------------------

bq. I would respectfully disagree about having redundant parameter checking. If it's not a
performance or maintenance burden, defence in depth against future stupidity is a Good Thing.

Yea, you're right.  I'd still like to see an implementation of something like IsValidPermissionMask()
that gets used in both places so if we change the bounds checks it can be done in 1 place
and doesn't require memorizing the upper bound.  I'm sure [~anatoli.shein] knows the details
better than myself at this point but IIRC that check needs to get done in some other RPC calls
as well. 

> libhdfs++: Implement chmod and chown
> ------------------------------------
>
>                 Key: HDFS-10524
>                 URL: https://issues.apache.org/jira/browse/HDFS-10524
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs-client
>            Reporter: Anatoli Shein
>            Assignee: Anatoli Shein
>         Attachments: HDFS-10524.HDFS-8707.000.patch
>
>




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

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-help@hadoop.apache.org


Mime
View raw message