[ 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)