GitHub user pauljackson opened a pull request:

    https://github.com/apache/tinkerpop/pull/547

    TINKERPOP-1589 Re-introduced CloseableIterator

    https://issues.apache.org/jira/browse/TINKERPOP-1589
    Add support for the closing of `Iterators` returned from
    `Vertex.vertices()` and `Vertex.edges()`.
    `Iterators` are closed once they are read to completion.
    Make `FlatMapStep` implement `AutoCloseable` in case iterator is not
    read to completion, should get closed when `Traversal` is closed.
    Remove unnecessary null check (null is never an instanceof).
    OLTP mode support only. More extensive changes required for OLAP.
    NOTE: Rethrowing checked Exception from `close()` as unchecked
    RuntimeException in order to retain `Step.reset()` and
    `AbstractStep.processNextStart()` signatures.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/pauljackson/tinkerpop tp32

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tinkerpop/pull/547.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #547
    
----
commit c8d5ff9db6ec8908abfad434c4fb339c5ad53468
Author: PaulJackson123 <pauljackson...@verizon.net>
Date:   2017-01-26T01:43:22Z

    TINKERPOP-1589 Re-introduced CloseableIterator
    
    https://issues.apache.org/jira/browse/TINKERPOP-1589
    Add support for the closing of Iterators returned from
    Vertex.vertices() and Vertex.edges().
    Iterators are closed once they are read to completion.
    Make FlatMapStep implement AutoCloseable in case iterator is not
    read to completion, should get closed when Traversal is closed.
    Remove unnecessary null check (null is never an instanceof).
    OLTP mode support only. More extensive changes required for OLAP.
    NOTE: Rethrowing checked Exception from close() as unchecked
    RuntimeException in order to retain Step.reset() and
    AbstractStep.processNextStart() signatured.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to