[GitHub] tinkerpop issue #521: TINKERPOP-1589 Re-introduced CloseableIterator

2017-01-05 Thread twilmes
Github user twilmes commented on the issue: https://github.com/apache/tinkerpop/pull/521 VOTE: +1 --- 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

[GitHub] tinkerpop issue #521: TINKERPOP-1589 Re-introduced CloseableIterator

2017-01-05 Thread lvca
Github user lvca commented on the issue: https://github.com/apache/tinkerpop/pull/521 VOTE +1 --- 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 fea

[GitHub] tinkerpop issue #521: TINKERPOP-1589 Re-introduced CloseableIterator

2017-01-05 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/521 VOTE +1 --- 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 fe

[GitHub] tinkerpop issue #521: TINKERPOP-1589 Re-introduced CloseableIterator

2017-01-05 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/521 I think I'm done - It's already implemented on `Traversal.close()`. I think it's good for vote. Don't think i'm missing anything. --- If your project is set up for it, you can reply to this em

[GitHub] tinkerpop issue #521: TINKERPOP-1589 Re-introduced CloseableIterator

2017-01-05 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/521 Nice. Clean backwards compatible `GraphStep.close()` method which uses `instanceof`. Are you looking for a VOTE now or are you still building on this PR? -- e.g. integrating it with `Traver