Merge branch 'TINKERPOP-1547' into tp32
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/d0389168 Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/d0389168 Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/d0389168 Branch: refs/heads/TINKERPOP-1529-variant2 Commit: d0389168d14bf8ce56918d8c639f3ab66176029a Parents: f448637 6bb8456 Author: Marko A. Rodriguez <okramma...@gmail.com> Authored: Thu Nov 10 17:29:56 2016 -0700 Committer: Marko A. Rodriguez <okramma...@gmail.com> Committed: Thu Nov 10 17:29:56 2016 -0700 ---------------------------------------------------------------------- CHANGELOG.asciidoc | 1 + .../optimization/RangeByIsCountStrategy.java | 55 +++++++++++--------- .../RangeByIsCountStrategyTest.java | 30 ++++++----- 3 files changed, 48 insertions(+), 38 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/d0389168/CHANGELOG.asciidoc ---------------------------------------------------------------------- diff --cc CHANGELOG.asciidoc index 0b765db,1f4e4cf..ccbbc89 --- a/CHANGELOG.asciidoc +++ b/CHANGELOG.asciidoc @@@ -26,8 -26,8 +26,9 @@@ image::https://raw.githubusercontent.co TinkerPop 3.2.4 (Release Date: NOT OFFICIALLY RELEASED YET) ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + * Added another optimization in `RangeByIsCountStrategy`, that removes `count().is()` altogether if it's not needed. * Fixed a OLAP `MatchStep.clone()`-bug that occurs when the `match()` is in a local child. +* Added another optimization in `RangeByIsCountStrategy`, that removes `count().is()` altogether if it's not needed. * Fixed a bug in `RangeByIsCountStrategy` where labeled parents shouldn't have the strategy applied to their children. * Fixed a bug in `PathRetractionStrategy` where `MatchEndStep` labels were being dropped when they shouldn't be. * Added `TinkerGraphCountStrategy` which translates `g.V().map*.count()` patterns into direct `Map.size()` calls in `TinkerGraph`.