[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 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.
---


[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 soon as 
`docker/build.sh -t -i` reports success.


---
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.
---


[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 through it here since it seem unrelated to 
this PR.


---
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.
---


[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 good.

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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---