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

Colin Patrick McCabe commented on HDFS-8924:
--------------------------------------------

* Suppress javac warning about typecast when loading ReplicaAccesorBuilder class
* Remove unused import line in {{ReplicaAccessorBuilder}} and 
{{ReplicaAccessor}}
* Fix some lines which were longer than 80 characters
* Remove change to {{ClientContext.java}}

bq. Would ReplicaAccessor, ReplicaAccessorBuilder to be interface?

Good question.  They are deliberately abstract classes rather than interfaces, 
so that we can add more methods later if needed (as long as we provide a 
default implementation in the base class)

> Add pluggable interface for reading replicas in DFSClient
> ---------------------------------------------------------
>
>                 Key: HDFS-8924
>                 URL: https://issues.apache.org/jira/browse/HDFS-8924
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: hdfs-client
>    Affects Versions: 2.8.0
>            Reporter: Colin Patrick McCabe
>            Assignee: Colin Patrick McCabe
>         Attachments: HDFS-8924.001.patch
>
>
> We should add a pluggable interface for reading replicas in the DFSClient.  
> This could be used to implement short-circuit reads on systems without file 
> descriptors, or for other optimizations.



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

Reply via email to