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

Brahma Reddy Battula commented on HDFS-13998:
---------------------------------------------

IMHO, HDFS-13732 change might not require..? As admin will be aware of 
configured policy and these are admin commands.

Adding RPC can mislead

    For concurrent calls and any error while getting the policy after setting.

and Extra overhead as [~ayushtkn] mentioned.

   Audit log ( for debugging) and RPC call

 If you all agree, we should revert HDFS-13732 before we ship 3.2 release and 
Follow up for next release..?

If we really required why can't we do through getserverdefaults()(by adding EC 
field there).

> 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