Github user jorgebay commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/710#discussion_r138317662
  
    --- Diff: 
gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Process/Traversal/DriverRemoteConnection/GraphTraversalTests.cs
 ---
    @@ -102,9 +102,9 @@ public void g_V_HasXname_markoX_ValueMap_Next()
                 var connection = _connectionFactory.CreateRemoteConnection();
                 var g = graph.Traversal().WithRemote(connection);
     
    -            var receivedValueMap = g.V().Has("name", 
"marko").ValueMap<string, object>().Next();
    +            var receivedValueMap = g.V().Has("name", 
"marko").ValueMap<object, object>().Next();
    --- End diff --
    
    The serialization format for maps changed (see JIRA tickets 1427 and 1658), 
the result from this traversal was originally a js object:
    
    ```js
    { "marko": { "@type": "g:Int32", "@value": 29 } }
    ```
    
    And changed in GraphSON3 to a type "g:Map" for which we use 
`IDictionary<object, object>`.
    
    I've started a discussion on the mailing list regarding collections child 
types: 
https://lists.apache.org/thread.html/3918353aaa63aa07b69214da24fa7aa0760004227fc57fa2b3bcae86@%3Cdev.tinkerpop.apache.org%3E


---

Reply via email to