[
https://issues.apache.org/jira/browse/HDFS-11182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15762551#comment-15762551
]
Arpit Agarwal commented on HDFS-11182:
--------------------------------------
Thanks for the feecback [~xyao]!
Pushed the following commit:
https://github.com/apache/hadoop/pull/168/commits/7ef55e7b2622de9345dd2d138c3e828ad5916bfa
- Removed the semaphore as both you and [~manju_hadoop] commented on it,
replaced with a simple counter initialized to the number of volumes.
- Factored out some common code in tests you pointed out into a {{@Before}}
method.
[~manju_hadoop], to address your other point about shared code between the sync
and async checks, there's a couple of differences that make it move it into a
common method.
> Update DataNode to use DatasetVolumeChecker
> -------------------------------------------
>
> Key: HDFS-11182
> URL: https://issues.apache.org/jira/browse/HDFS-11182
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: datanode
> Reporter: Arpit Agarwal
> Assignee: Arpit Agarwal
>
> Update DataNode to use the DatasetVolumeChecker class introduced in
> HDFS-11149 to parallelize disk checks.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
