Skip to content

TINKERPOP-1681 Multiple hasId's are or'd into GraphStep#617

Merged
asfgit merged 1 commit intotp32from
TINKERPOP-1681
Jun 1, 2017
Merged

TINKERPOP-1681 Multiple hasId's are or'd into GraphStep#617
asfgit merged 1 commit intotp32from
TINKERPOP-1681

Conversation

@dkuppitz
Copy link
Contributor

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

Fixed folding of multiple hasId()'s into GraphStep.

VOTE: +1

@okram
Copy link
Contributor

okram commented May 31, 2017

Seems right to me. Just to be certain x.y.z are ANDs and before, for hasLabel() and hasId() they were being OR'd. Test cases make sense.

VOTE +1.

@spmallette
Copy link
Contributor

Needs an entry to CHANGELOG, but all tests pass with docker/build.sh -t -n -i

VOTE +1

@asfgit asfgit merged commit 12aa2d2 into tp32 Jun 1, 2017
@asfgit asfgit deleted the TINKERPOP-1681 branch June 23, 2017 12:16
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.

4 participants