[GitHub] tinkerpop issue #483: TINKERPOP-1547: Two bugs found associated with MatchSt...

2016-11-10 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/483 Changes pushed. `docker/build.sh -t -i -n` succeeded. 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 pro

[GitHub] tinkerpop issue #483: TINKERPOP-1547: Two bugs found associated with MatchSt...

2016-11-10 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/483 @dkuppitz -- note that @spmallette never VOTE'd so when you are done with your testing and updates, just VOTE and I will merge. --- If your project is set up for it, you can reply to this email and

[GitHub] tinkerpop issue #483: TINKERPOP-1547: Two bugs found associated with MatchSt...

2016-11-10 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/483 Please don't merge this PR yet. While testing the changes in `RangeByIsCountStrategy` I realized that we can add a minor tweak to get another great optimization. I'll push my local changes as soo

[GitHub] tinkerpop issue #483: TINKERPOP-1547: Two bugs found associated with MatchSt...

2016-11-10 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/483 @twilmes can you please send some specifics on what interruption tests were tanking on you and under what circumstances? Maybe just ping me on hipchat when you have a moment rather than go thro

[GitHub] tinkerpop issue #483: TINKERPOP-1547: Two bugs found associated with MatchSt...

2016-11-09 Thread twilmes
Github user twilmes commented on the issue: https://github.com/apache/tinkerpop/pull/483 Been having problems with traversal interrupted tests failing across many branches but I ran other tests including neo4j and everything passed. Checked out match retraction updates and they look