Right, I updated my comment right after I made it, when I noticed the
difference.
I shouldn't think it matters which one to keep. "LazyIterator" is a little
shorter to write. :)
There are a number of other Iterators (noted in the comments to that ticket)
that seem to be depreciate-able. E.g. SingletonIterator has equivalent Guava
functionality, and UniqueExtendedIterator has its own comments suggesting that
it be deprecated ("New development should use use <code>UniqueFilter</code>…").
As I said in an earlier message, I will issue a reworked PR #79 with those
suggestions, but I will not touch the lazy iterators.
---
A. Soroka
The University of Virginia Library
On Jun 24, 2015, at 11:32 AM, Claude Warren <[email protected]> wrote:
> Yes. LazyIterator implements ExtendedIterator. LateBindingIterator
> implements Iterator
>
> My plan -- probably won't execute until tomorrow night -- is to complete
> the implementation of LazyIterator for both (2.13.1 and 3.0.0) and then
> deprecate LateBinding in favor of Lazy as ExtendedIterator implements
> Iterator.
>
> Though I could very easily be swayed to alter LateBindingIterator to
> implement ExtendedIterator and deprecate LazyIterator.
>
> Claude
>
> On Wed, Jun 24, 2015 at 4:13 PM, A. Soroka (JIRA) <[email protected]> wrote:
>
>>
>> [
>> https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14599547#comment-14599547
>> ]
>>
>> A. Soroka commented on JENA-966:
>> --------------------------------
>>
>> Is there any difference between {{LateBindingIterator}} and
>> {{LazyIterator}}?
>>
>>> 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)
>>
>
>
>
> --
> I like: Like Like - The likeliest place on the web
> <http://like-like.xenei.com>
> LinkedIn: http://www.linkedin.com/in/claudewarren
