[
https://issues.apache.org/jira/browse/HDFS-14188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16767210#comment-16767210
]
Kitti Nanasi commented on HDFS-14188:
-------------------------------------
Thanks for the review [~ayushtkn]!
* It's a good idea to save efforts by running the verify for multiple policies
at the same time, I implemented it in patch v003.
* About the test, although it would same some execution time to merge the
tests together, I think it would be less readable and I would have to clear the
system out inside the method multiple times, instead of leaving it for the
tearDown method, which would be a bit confusing. But I am not against merging
them if you want to save on test execution time, so let me know your opinion on
that.
* Thanks for reminding me about the documentation, I extended it.
> Make hdfs ec -verifyClusterSetup command accept an erasure coding policy as a
> parameter
> ---------------------------------------------------------------------------------------
>
> Key: HDFS-14188
> URL: https://issues.apache.org/jira/browse/HDFS-14188
> Project: Hadoop HDFS
> Issue Type: Improvement
> Components: erasure-coding
> Affects Versions: 3.1.1
> Reporter: Kitti Nanasi
> Assignee: Kitti Nanasi
> Priority: Major
> Attachments: HDFS-14188.001.patch, HDFS-14188.002.patch,
> HDFS-14188.003.patch
>
>
> hdfs ec -verifyClusterSetup command verifies if there are enough data nodes
> and racks for the enabled erasure coding policies
> I think it would be beneficial if it could accept an erasure coding policy as
> a parameter optionally. For example the following command would run the
> verify for only the RS-6-3-1024k policy.
> {code:java}
> hdfs ec -verifyClusterSetup -policy RS-6-3-1024k
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
