GitHub user dkuppitz opened a pull request:
https://github.com/apache/incubator-tinkerpop/pull/320
TINKERPOP-1312 .count().is(0) is not properly optimized
Tweaked `RangeByIsCountStrategy`. The patterns ...
* `....count().is(0)`
* `....count().is(lt(1))`
* `....count().is(lte(0))`
... are now replaced by `not(...)`.
`docker/build.sh -t -i -n` succeeded.
VOTE: +1
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/incubator-tinkerpop TINKERPOP-1312
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-tinkerpop/pull/320.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 #320
----
commit 5f3305315052258d3e659b1d984e87e73ad2b0af
Author: Daniel Kuppitz <[email protected]>
Date: 2016-05-25T19:13:48Z
Tweaked `RangeByIsCountStrategy`. The pattern `outE().count().is(0)` is now
replaced by `not(outE())`
commit 5f5a0f37221c0ca7ec777418b96429b4a20d9934
Author: Daniel Kuppitz <[email protected]>
Date: 2016-05-25T19:24:58Z
Updated CHANGELOG
commit 5751456fcc91ea4ab9b7628e91209b96ea5d958e
Author: Daniel Kuppitz <[email protected]>
Date: 2016-05-25T19:33:44Z
Increase step offset by 2 if `foo.count().is(0)` was replaced by `not(foo)`.
commit 3f9a256db449f0a8bbca3ca9fce8355a6dbb9a39
Author: Daniel Kuppitz <[email protected]>
Date: 2016-05-25T19:55:27Z
Fixed the previous commit. We don't want to increase the step offset, but
decrease the traversal size, after removing steps from a traversal.
----
---
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.
---
