GitHub user spmallette opened a pull request:
https://github.com/apache/incubator-tinkerpop/pull/143
TINKERPOP3-955 Registered HashMap$Node to GryoMapper.
https://issues.apache.org/jira/browse/TINKERPOP3-955
Note that this class had to be registered via reflection as it is a private
class of HashMap. Added tests to validate that the change works with the code
specified in the issue.
Please review as it pertains to area of serialization that you are familiar
with. To test with gremlin-server you need to run the integration tests:
```text
mvn clean install
mvn verify -pl gremlin-server -DskipIntegrationTests=false
```
VOTE: +1
btw, we _could_ postpone this to 3.1.1 if we wanted - i think. there is a
workaround to the bug in that the person just doesn't have to `next()` the
result to get it to serialize.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/incubator-tinkerpop TINKERPOP3-955
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-tinkerpop/pull/143.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 #143
----
commit 8b70c5cf81ac0fbd5246489c91ef864ae2247495
Author: Stephen Mallette <[email protected]>
Date: 2015-11-11T16:42:44Z
TINKERPOP3-955 Registered HashMap$Node to GryoMapper.
Note that this class had to be registered via reflection as it is a private
class of HashMap. Added tests to validate that the change works with the code
specified in the issue.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
