[
https://issues.apache.org/jira/browse/TINKERPOP-1434?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15630562#comment-15630562
]
ASF GitHub Bot commented on TINKERPOP-1434:
-------------------------------------------
GitHub user spmallette opened a pull request:
https://github.com/apache/tinkerpop/pull/479
TINKERPOP-1434 Block calls on "remote" traversal to get side-effects
https://issues.apache.org/jira/browse/TINKERPOP-1434
This prevents calls for side-effects that are not yet ready on the server.
This was generally a problem in tests, but users could have also run afoul of
the issue if they decided to get low-level with the API. It is more likely that
they would grab side-effects with a call to `cap()` so this would be a
non-issue in that case.
Did multiple runs of `mvn clean install && mvn verify -pl gremlin-server
-DincludeNeo4j -DskipIntegrationTests=false` and it passes nicely. I never saw
the test failures that triggered the effort to produce a fix for this problem,
so @dkuppitz who saw it regularly should give this a review. Also, if travis
passes, that would be a good sign - the error was seen there sometimes as well.
Please be sure to use `-DincludeNeo4j` if re-running the tests.
VOTE +1
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/tinkerpop TINKERPOP-1434
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tinkerpop/pull/479.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 #479
----
commit 46a61bb124d27aebcf96a639b2614e16389b02f9
Author: Stephen Mallette <[email protected]>
Date: 2016-11-02T21:12:36Z
TINKERPOP-1434 Block calls on "remote" traversal to get side-effects
This prevents calls for side-effects that are not yet ready on the server.
This was generally a problem in tests, but usrs could have also run afoul of
the issue if they decided to get low-level with the API. It is more likely that
they would grab side-effects with a call to cap() so this would be a non-issue
in that case.
----
> Block calls to traversal side-effects until read is complete
> ------------------------------------------------------------
>
> Key: TINKERPOP-1434
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1434
> Project: TinkerPop
> Issue Type: Improvement
> Components: driver
> Affects Versions: 3.2.2
> Reporter: stephen mallette
> Assignee: stephen mallette
> Priority: Minor
>
> Might make usability a little better if calls to {{getSideEffects()}} could
> block until the remote {{Traversal}} complete iteration (i.e. on "read
> complete") - or perhaps they return a {{Future}} or something.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
