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

Tsz Wo Nicholas Sze commented on HDFS-4114:
-------------------------------------------

I see.  What you want actually is some kind of interface for namenode, like 
FsDatasetSpi for datanode.  I think it is a good idea.

FsDatasetSpi does have ExternalDatasetImpl (as a test) in order to guarantee 
that all the classes necessary to implement the interface are (and remain) 
publicly accessible.  How about we keep on the method signature and remove all 
the BackupNode implementation, i.e. keep empty methods as ExternalDatasetImpl?  
Then, it is much easier to maintain.

> Remove the BackupNode and CheckpointNode from trunk
> ---------------------------------------------------
>
>                 Key: HDFS-4114
>                 URL: https://issues.apache.org/jira/browse/HDFS-4114
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: Eli Collins
>            Assignee: Suresh Srinivas
>         Attachments: HDFS-4114.000.patch, HDFS-4114.001.patch, HDFS-4114.patch
>
>
> Per the thread on hdfs-dev@ (http://s.apache.org/tMT) let's remove the 
> BackupNode and CheckpointNode.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to