[
https://issues.apache.org/jira/browse/HADOOP-1200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12493796
]
Hairong Kuang commented on HADOOP-1200:
---------------------------------------
> You have replaced 'shutdown(); throw iex;' with 'checkDiskError(iex); throw
> iex;' does not shutdown if checkDirs() does not throw DiskErrorException. Is
> this functionality change intentional?
Yes, the idea is that we do not shutdown datanode if IOException is caused by
any temporary error.
> Another functionality change is that data.invalidate(toDelete)'s exception is
> ignored in processCommand(). This change is probably not necessary since
> offerService() already handles the exception.
You are right. The new patch will rethrow the exception.
> Datanode should periodically do a disk check
> --------------------------------------------
>
> Key: HADOOP-1200
> URL: https://issues.apache.org/jira/browse/HADOOP-1200
> Project: Hadoop
> Issue Type: Bug
> Components: dfs
> Affects Versions: 0.12.2
> Reporter: Hairong Kuang
> Assigned To: Hairong Kuang
> Priority: Blocker
> Fix For: 0.13.0
>
> Attachments: diskCheck.patch
>
>
> HADOOP-1170 removed the disk checking feature. But this is a needed feature
> for maintaining a large cluster. I agree that checking the disk on every I/O
> is too costly. A nicer approach is to have a thread that periodically do a
> disk check. It then automatically decommissions itself when any error occurs.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
