Skip to content

Conversation

@spmallette
Copy link
Contributor

https://issues.apache.org/jira/browse/TINKERPOP-1870

IndexedTraverserSet does what VertexTraverserSet was doing, but in a more complete way. It respects all the semantics of a TraverserSet and fixes a problem that VertexTraverserSet had where it didn't properly account for Traverser merges. Added a full grip of unit tests for both TraverserSet and IndexedTraverserSet.

All tests pass with docker/build.sh -t -i

VOTE +1

…erSet

IndexedTraverserSet does what VertexTraverserSet was doing, but in a more complete way. It respects all the semantics of a TraverserSet and fixes a problem that VertexTraverserSet had where it didn't properly account for Traverser merges. Added a full grip of unit tests for both TraverserSet and IndexedTraverserSet.
@artem-aliev
Copy link

+1
I reran my small perf test and get the same result with this new structure, but it looks more solid and inline with other code.

@dkuppitz
Copy link
Contributor

VOTE: +1

@robertdale
Copy link
Member

VOTE +1

@asfgit asfgit merged commit 068bef8 into tp32 Jan 24, 2018
@asfgit asfgit deleted the TINKERPOP-1870 branch April 14, 2018 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants