[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14647612#comment-14647612
]
ASF GitHub Bot commented on JENA-966:
-
Github user ajs6f commented on a diff in the pull
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14646710#comment-14646710
]
ASF GitHub Bot commented on JENA-966:
-
Github user ajs6f commented on a diff in the pull
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14627974#comment-14627974
]
ASF GitHub Bot commented on JENA-966:
-
Github user ajs6f closed the pull request at:
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14624531#comment-14624531
]
ASF subversion and git services commented on JENA-966:
--
Commit
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14623937#comment-14623937
]
ASF subversion and git services commented on JENA-966:
--
Commit
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14613322#comment-14613322
]
ASF subversion and git services commented on JENA-966:
--
Commit
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14608023#comment-14608023
]
ASF subversion and git services commented on JENA-966:
--
Commit
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14608025#comment-14608025
]
Andy Seaborne commented on JENA-966:
Option 1 used to DRY the code.
LazyIterator
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14608035#comment-14608035
]
ASF subversion and git services commented on JENA-966:
--
Commit
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14604610#comment-14604610
]
Andy Seaborne commented on JENA-966:
There is a quicker build for testing the main
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14604606#comment-14604606
]
Andy Seaborne commented on JENA-966:
The Jenkins test job has got failures:
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14604886#comment-14604886
]
Andy Seaborne commented on JENA-966:
In {{Unparser.listSubjects}} the iterator
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14604892#comment-14604892
]
ASF subversion and git services commented on JENA-966:
--
Commit
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14603311#comment-14603311
]
ASF subversion and git services commented on JENA-966:
--
Commit
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14603345#comment-14603345
]
ASF subversion and git services commented on JENA-966:
--
Commit
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14603365#comment-14603365
]
ASF subversion and git services commented on JENA-966:
--
Commit
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14603372#comment-14603372
]
Claude Warren commented on JENA-966:
I think we should deprecate them now and remove
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14603369#comment-14603369
]
ASF subversion and git services commented on JENA-966:
--
Commit
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14603324#comment-14603324
]
Andy Seaborne commented on JENA-966:
Let's delete unused LateBindingIterator and
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14603448#comment-14603448
]
Andy Seaborne commented on JENA-966:
Not critical either way but, yes, that's pretty
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14601203#comment-14601203
]
A. Soroka commented on JENA-966:
Okay, I had that misapprehension based on some comments you
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14600155#comment-14600155
]
Andy Seaborne commented on JENA-966:
There is no strategy to move
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14599460#comment-14599460
]
A. Soroka commented on JENA-966:
That's very true-- I should not have been so brash about
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14599547#comment-14599547
]
A. Soroka commented on JENA-966:
Is there any difference between {{LateBindingIterator}} and
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14599624#comment-14599624
]
A. Soroka commented on JENA-966:
As per comment on the dev mailing list, [~claudenw] is
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.
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14599637#comment-14599637
]
A. Soroka commented on JENA-966:
[~andy.seaborne], I am still not clear about your objection
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
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14599851#comment-14599851
]
ASF GitHub Bot commented on JENA-966:
-
GitHub user ajs6f opened a pull request:
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14599856#comment-14599856
]
ASF GitHub Bot commented on JENA-966:
-
Github user ajs6f commented on the pull request:
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14599857#comment-14599857
]
ASF GitHub Bot commented on JENA-966:
-
Github user ajs6f closed the pull request at:
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14599853#comment-14599853
]
ASF GitHub Bot commented on JENA-966:
-
Github user ajs6f commented on the pull request:
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14598364#comment-14598364
]
Andy Seaborne commented on JENA-966:
The contract for {{Supplier}} is not specific
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14597501#comment-14597501
]
A. Soroka commented on JENA-966:
I'm happy to leave this to y'all. I understand the desire
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596485#comment-14596485
]
A. Soroka commented on JENA-966:
I would rather actually remove the {{LazyIterator}} class
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596545#comment-14596545
]
A. Soroka commented on JENA-966:
Just as a sidenote for anyone following this who is not
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596710#comment-14596710
]
Andy Seaborne commented on JENA-966:
Adding {{andThen(SupplierIteratorT)}} to
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596588#comment-14596588
]
A. Soroka commented on JENA-966:
[~claudenw], would it be useful to you for me to send a PR
My thought was to implement the methods mentioned in the bug using the
NiceIterator. and leave the create() as abstract so one could simply
implement create() and be done. And with one simple method it could be
impelemented in an anonymous class ala Map1.
Claude
On Mon, Jun 22, 2015 at 10:48
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596500#comment-14596500
]
Claude Warren commented on JENA-966:
I don't see how to quickly implement LazyIterator
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596530#comment-14596530
]
A. Soroka commented on JENA-966:
There's a few points here:
- I have _not _suggested
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596509#comment-14596509
]
Claude Warren commented on JENA-966:
My other concern is the distance between 2.13 and
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596501#comment-14596501
]
A. Soroka commented on JENA-966:
I'm suggesting that I'd rather remove {{LazyIterator}} and
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14596484#comment-14596484
]
Claude Warren commented on JENA-966:
This bug is for the LazyIterator. I think we
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14591752#comment-14591752
]
A. Soroka commented on JENA-966:
Yes. Is that problematic? Isn't the larger strategy to
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14591746#comment-14591746
]
A. Soroka commented on JENA-966:
I don't think iterator refactoring is what is being
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590304#comment-14590304
]
ASF GitHub Bot commented on JENA-966:
-
GitHub user ajs6f opened a pull request:
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590306#comment-14590306
]
A. Soroka commented on JENA-966:
The PR above actually removes the types in question. If
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590674#comment-14590674
]
Andy Seaborne commented on JENA-966:
{{NiceIterator}} now uses {{Iter.map}} via
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590523#comment-14590523
]
Andy Seaborne commented on JENA-966:
I agree - delete it.
LazyIterator
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590522#comment-14590522
]
Andy Seaborne commented on JENA-966:
I don't think that is a good idea.
LazyIterator
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590530#comment-14590530
]
A. Soroka commented on JENA-966:
Re: `Map1Iterator`, I didn't connect it to `Iter::map`, I
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590570#comment-14590570
]
Andy Seaborne commented on JENA-966:
Why is iterator refactoring important?
(and Java8
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14589458#comment-14589458
]
Andy Seaborne commented on JENA-966:
There are no uses of this class in the codebase
I wanted to use it in an application. Is there a replacement?
On Wed, Jun 17, 2015 at 9:10 AM, Andy Seaborne (JIRA) j...@apache.org
wrote:
[
On 17/06/15 10:07, Claude Warren wrote:
I wanted to use it in an application. Is there a replacement?
I just found IteratorDelayedInitialization in jena-base.
What also springs to mind is looking at the iterators in Commons. They
added forms at your suggestion of looking at jena-core
I think there are interesting iterators in Commons Collections and some utility
code in Commons Functor (unreleased) too.
HTH
Bruno
From: Andy Seaborne a...@apache.org
To: dev@jena.apache.org
Sent: Wednesday, June 17, 2015 9:50 PM
Subject: Re: [jira] [Commented] (JENA-966) LazyIterator
How about using a Java 8 SupplierIteratorT? That's pretty lazy.
---
A. Soroka
The University of Virginia Library
On Jun 17, 2015, at 5:07 AM, Claude Warren cla...@xenei.com wrote:
I wanted to use it in an application. Is there a replacement?
On Wed, Jun 17, 2015 at 9:10 AM, Andy Seaborne
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590002#comment-14590002
]
A. Soroka commented on JENA-966:
There is also {{LateBindingIterator}} which is only used by
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590012#comment-14590012
]
A. Soroka commented on JENA-966:
I think it could be argued that {{Map1Iterator}} should be
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590036#comment-14590036
]
A. Soroka commented on JENA-966:
{{SingletonIterator}} could be plausibly be deprecated in
[
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14590038#comment-14590038
]
A. Soroka commented on JENA-966:
{{UniqueExtendedIterator}} is not used anywhere, and even
62 matches
Mail list logo
