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

Mike Drob commented on ACCUMULO-2486:
-------------------------------------

Yea, I was reading something unrelated about using covariant return types for 
{{clone()}} and this seemed similar enough to warrant a discussion. Another 
nice side effect is that this could potentially clean up the mess that is 
{{RangeIterator}} (and likely others).

> Use covariant return types in SKVI.deepCopy
> -------------------------------------------
>
>                 Key: ACCUMULO-2486
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2486
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Mike Drob
>             Fix For: 1.7.0
>
>
> Starting with Java 5, subclasses with method overrides can declare covariant 
> return types. In general, I believe that we want to do this, as it provides 
> stronger type guarantees with not a lot of extra cost.
> Example:
> {code}
> public interface SortedKeyValueIterator {
>       SortedKeyValueIterator deepCopy(IteratorEnvironment env);
> }
> public interface InterruptibleIterator extends SortedKeyValueIterator {
>       @Override
>       InterruptibleIterator deepCopy(IteratorEnvironment env);
> }
> {code}
> I believe this change will be fully backwards compatible with the existing 
> APIs.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to