[ 
https://issues.apache.org/jira/browse/HDFS-11426?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Wang updated HDFS-11426:
-------------------------------
    Attachment: HDFS-11426.001.patch

Patch attached, I borrowed heavily from StoragePolicyAdmin. I think the SP APIs 
isn't ideal (IMO it's better to use positional args for required arguments), 
but uniformity is better yet.

There are some additional changes I'm planning as follow-ons, like changing the 
setECPolicy API to take a string rather than a byte (which pushes the 
validation to the NN, and makes it more similar to the SP set API), and also 
making the EC policy required in {{set}}'s PB as groundwork for removing the 
concept of the system default EC policy.

> Refactor EC CLI to be similar to storage policies CLI
> -----------------------------------------------------
>
>                 Key: HDFS-11426
>                 URL: https://issues.apache.org/jira/browse/HDFS-11426
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: erasure-coding, shell
>    Affects Versions: 3.0.0-alpha1
>            Reporter: Andrew Wang
>            Assignee: Andrew Wang
>              Labels: hdfs-ec-3.0-must-do
>         Attachments: HDFS-11426.001.patch
>
>
> The {{hdfs erasurecode}} CLI is similar to {{hdfs storagepolicies}} in terms 
> of functionality, but different in terms of behavior. Let's refactor 
> {{ECCli}} to be more similar to the various Admin classes we already have, 
> and also make its calling syntax mimic {{hdfs storagepolicies}} as closely as 
> possible.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to