GitHub user sheldonkhall opened a pull request:
https://github.com/apache/tinkerpop/pull/632
TINKERPOP-1519: tinker graph computer does not handle multiple scopes
https://issues.apache.org/jira/browse/TINKERPOP-1519
This change modifies the tinker graph computer so that each message sent in
a vertex program remembers its scope. Previously when the receiveMessages
method on TinkerMessenger was called it would loop through ALL message scopes
and then ALL messages, which is incorrect. Now the method loops over each
scope, and then each message within that scope.
I have added the regression test suggested in the JIRA ticket and run mvn
clean install locally to confirm everything passes OK.
I had a quick look at the latest master, and this bugfix may need to be
merged there too.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sheldonkhall/tinkerpop tp31
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tinkerpop/pull/632.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 #632
----
commit 28c514da9065683ed90ea6aabc66ffdcbab99c11
Author: Sheldon <[email protected]>
Date: 2017-06-15T17:00:32Z
bugfix
commit 99679a037db3e86112471a40cd454114231852b6
Author: Sheldon <[email protected]>
Date: 2017-06-19T09:35:16Z
add the regression test
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
