Github user jorgebay commented on the issue:
https://github.com/apache/tinkerpop/pull/710
Rebased onto master, tests pass with `mvn clean install -P gremlin-dotnet`.
VOTE +1
I'll merge it once CI jobs finish.
---
Github user spmallette commented on the issue:
https://github.com/apache/tinkerpop/pull/710
Unfortunately, you'll need to rebase because the Gremlin Server test server
script moved out of the pom.xml, but your changes are easy enough.
All tests pass with `docker/build.sh -t -n
Github user FlorianHockmann commented on the issue:
https://github.com/apache/tinkerpop/pull/710
VOTE: +1
---
Github user jorgebay commented on the issue:
https://github.com/apache/tinkerpop/pull/710
Thanks @FlorianHockmann for the feedback.
I've addressed the issues you mentioned.
About `ValueMap()`, we can continue the discussion on the
mailing list:
https://lists.apache.org/th
