GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/535
TINKERPOP-1601 LazyBarrierStrategy should not check is.testing
https://issues.apache.org/jira/browse/TINKERPOP-1601
Only a minor improvement (strategies shouldn't behave differently depending
on whether we're in testing mode or not). If a test requires that certain
strategies don't do anything, then it's the test's job to exclude those
strategies.
`docker/build.sh -t -i` succeeded.
VOTE: +1
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/tinkerpop TINKERPOP-1601
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tinkerpop/pull/535.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 #535
----
commit 0edf2e929b244055720d0e2054f21caef2205cc1
Author: Daniel Kuppitz <[email protected]>
Date: 2017-01-16T19:21:24Z
Removed IS_TESTING check from LazyBarrierStrategy.
commit 3179f6dab9065206a811e124c7827d4d9c5211ad
Author: Daniel Kuppitz <[email protected]>
Date: 2017-01-17T01:43:41Z
fixed assertion
commit ff844904df2b9e5d48c4725e8679f139c0e463e0
Author: Daniel Kuppitz <[email protected]>
Date: 2017-01-17T01:44:13Z
Changed btecode representation of excluded strategies to prevent
serialization issues.
----
---
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.
---
