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

Kitti Nanasi commented on HDFS-14061:
-------------------------------------

Thanks for the review [~jojochuang]!

I will clean up the log messages in HDFS-14125, so for now I will leave them as 
is. I addressed your comments in the newest patch.

About using all DataNodes in the verification, we did not consider in 
-HDFS-12946- to use only a subset of all DataNodes, that would make the 
verification a bit stricter and I am fine with that as well, though it would 
not make much difference in my opinion. Let me know if you have any suggestions 
on that.

> Check if the cluster topology supports the EC policy before setting, enabling 
> or adding it
> ------------------------------------------------------------------------------------------
>
>                 Key: HDFS-14061
>                 URL: https://issues.apache.org/jira/browse/HDFS-14061
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: erasure-coding, hdfs
>    Affects Versions: 3.1.1
>            Reporter: Kitti Nanasi
>            Assignee: Kitti Nanasi
>            Priority: Major
>         Attachments: HDFS-14061.001.patch, HDFS-14061.002.patch, 
> HDFS-14061.003.patch, HDFS-14061.004.patch, HDFS-14061.005.patch
>
>
> HDFS-12946 introduces a command for verifying if there are enough racks and 
> datanodes for the enabled erasure coding policies.
> This verification could be executed for the erasure coding policy before 
> enabling, setting or adding it and a warning message could be written if the 
> verify fails, or the policy setting could be even failed in this case.



--
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