[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14601203#comment-14601203
]
A. Soroka commented on JENA-966:
--------------------------------
Okay, I had that misapprehension based on some comments you ([~andy.seaborne])
made during the introduction of Java 8 syntax a while ago. Clearly I
misunderstood you, for which I apologize. I certainly am _not_ trying to
introduce "Changes for change sake"-- no one has enough time for that,
especially on an unpaid basis. {grin}
[~andy.seaborne] or [~claudenw], can I ask you to clarify when it is
appropriate to use {{ExtendedIterator}} and when it is better to use {{Iter}}?
{{Iter}} features all of the functionality of {{ExtendedIterator}} and more.
Since {{Iter}} is in {{jena-base}} and {{ExtendedIterator}} is in
{{jena-core}}, {{Iter}} will be available any time {{ExtendedIterator}} is.
That's another reason I assumed we would want to shift to {{Iter}}-- from a
naive point of view, they look like flat-out duplication. If you can unpack
this a bit for me, I will send a PR with some Javadocs additions for both that
will prevent anyone else from going down the mistaken road I just wasted my
time on.
> 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)
