[
https://issues.apache.org/jira/browse/TINKERPOP-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16472864#comment-16472864
]
ASF GitHub Bot commented on TINKERPOP-1958:
-------------------------------------------
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/864
TINKERPOP-1958 Fixed a bug in TinkerGraphCountStrategy
https://issues.apache.org/jira/browse/TINKERPOP-1958
The strategy did not consider, that certain map steps may not emit an
element.
`docker/build.sh -t -i` passed.
VOTE: +1
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/tinkerpop TINKERPOP-1958
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tinkerpop/pull/864.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 #864
----
commit da5b7da2c4dd7202b69f2a211542e9aa711fd2d8
Author: Daniel Kuppitz <daniel_kuppitz@...>
Date: 2018-05-09T14:53:24Z
TINKERPOP-1958 Fixed a bug in TinkerGraphCountStrategy
The strategy did not consider, that certain map steps may not emit an
element.
----
> TinkerGraphCountStrategy can return wrong counts
> ------------------------------------------------
>
> Key: TINKERPOP-1958
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1958
> Project: TinkerPop
> Issue Type: Bug
> Components: process
> Affects Versions: 3.2.8, 3.3.2
> Reporter: Daniel Kuppitz
> Assignee: Daniel Kuppitz
> Priority: Major
>
> {noformat}
> gremlin> g = TinkerFactory.createModern().traversal()
> ==>graphtraversalsource[tinkergraph[vertices:6 edges:6], standard]
> gremlin> g.V().map(out())
> ==>v[3]
> ==>v[5]
> ==>v[3]
> gremlin> g.V().map(out()).count()
> ==>6
> gremlin> g.V().select("a")
> gremlin> g.V().select("a").count()
> ==>6
> {noformat}
> Not tested in 3.2.x, but it's probably the same issue.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
