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

Claude Warren commented on JENA-966:
------------------------------------

What say we open an epic to cover iterator refactoring.

My opinion is that it should deprecate all (most?) iterators and point devs to 
the java8 replacements.

I think we should deprecate since change iterators in legacy code might be a 
big step -- well beyond renaming included packages.

As for this defect.  I think that the methods noted above should be added to 
the iterator both in 3.0.0 and in 2.13.1 (or whatever the latest 2.x version 
is).

Thoughts?


> LazyIterator
> ------------
>
>                 Key: JENA-966
>                 URL: https://issues.apache.org/jira/browse/JENA-966
>             Project: Apache Jena
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: Jena 3.0.0
>            Reporter: Claude Warren
>            Assignee: Claude Warren
>
> LazyIterator is an abstract class.  The documentation indicates that the 
> create() method needs to be overridden to create an instance.  From this I 
> would expect that 
> now LazyIterator(){
> @Override
> public ExtendedIterator<Model> create() {
>                       ...
> }};
> Would work however LazyIterator does not override:
> remoteNext(), andThen(), toList(), and toSet().
> I believe these should be implemented in the class.



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

Reply via email to