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

Xiao Chen commented on HDFS-13998:
----------------------------------

bq.  admin commands.
-setPolicy is _not_ an admin command. Users who have write perm can set policy, 
similar to setxattr. IMO this is the most confusing part - as a user you can 
set policy, but if you set with default you don't know what's the default and 
have to explicitly get.


bq. incompatible
That's an interesting question. I went to check the compat guideline, it does 
mention
{quote}
All Hadoop CLI paths, usage, and output SHALL be considered Public and Stable 
unless documented as experimental and subject to change.

Note that the CLI output SHALL be considered distinct from the log output 
generated by the Hadoop CLIs. The latter SHALL be governed by the policy on log 
output. Note also that for CLI output, all changes SHALL be considered 
incompatible changes.
{quote}
I was under the impression that the explicit {{InterfaceAudience.Private}} on 
the ECAdmin class means we can change it. It seems the compat guideline for CLI 
should take precedence since the CLI isn't marked experimental. So..... (see 
below)

bq. If you all agree, we should revert HDFS-13732 before we ship 3.2 release 
and Follow up for next release..?
I'll go ahead and revert HDFS-13732 entirely. Will leave it up for discussion 
on how we do this on trunk.

> ECAdmin NPE with -setPolicy -replicate
> --------------------------------------
>
>                 Key: HDFS-13998
>                 URL: https://issues.apache.org/jira/browse/HDFS-13998
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: erasure-coding
>    Affects Versions: 3.2.0, 3.1.2
>            Reporter: Xiao Chen
>            Assignee: Zsolt Venczel
>            Priority: Major
>         Attachments: HDFS-13998.01.patch, HDFS-13998.02.patch, 
> HDFS-13998.03.patch
>
>
> HDFS-13732 tried to improve the output of the console tool. But we missed the 
> fact that for replication, {{getErasureCodingPolicy}} would return null.
> This jira is to fix it in ECAdmin, and add a unit test.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to