[ http://issues.apache.org/jira/browse/HADOOP-321?page=comments#action_12417893 ]
Doug Cutting commented on HADOOP-321: ------------------------------------- I like this. However we cannot simply remove DataNodeReport, since it's public, and may be referenced by user code. Rather, we should change it to a deprecated subclass of DatanodeInfo that defines no methods, and file a bug to remove it from the next release. Also, DatanodeInfo subclasses the non-public DatanodeID, so that class should probably be made public too. > DatanodeInfo refactoring > ------------------------ > > Key: HADOOP-321 > URL: http://issues.apache.org/jira/browse/HADOOP-321 > Project: Hadoop > Type: Improvement > Components: dfs > Reporter: Konstantin Shvachko > Attachments: DatanodeInfo_refactor.patch > > I'm trying to refactor some name node classes, which seem to be similar. > So DatanodeInfo is a public api now for purely external ( to name node) use. > The name node class that stores information about data nodes including the > set of its blocks is called DatanodeDescriptor. > The DatanodeReport is removed since it was a variation of DatanodeInfo. > Previously DatanodeInfo and DatanodeDescriptor were the same class, and > DatanodeReport was used for reporting node statistics only. > This is a preparation step for HADOOP-306. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
