[ 
https://issues.apache.org/jira/browse/TINKERPOP-1730?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16161506#comment-16161506
 ] 

ASF GitHub Bot commented on TINKERPOP-1730:
-------------------------------------------

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

    https://github.com/apache/tinkerpop/pull/710#discussion_r138109326
  
    --- Diff: 
gremlin-dotnet/src/Gremlin.Net/Structure/IO/GraphSON/Path3Deserializer.cs ---
    @@ -0,0 +1,45 @@
    +#region License
    +
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one
    + * or more contributor license agreements.  See the NOTICE file
    + * distributed with this work for additional information
    + * regarding copyright ownership.  The ASF licenses this file
    + * to you under the Apache License, Version 2.0 (the
    + * "License"); you may not use this file except in compliance
    + * with the License.  You may obtain a copy of the License at
    + *
    + *     http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing,
    + * software distributed under the License is distributed on an
    + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    + * KIND, either express or implied.  See the License for the
    + * specific language governing permissions and limitations
    + * under the License.
    + */
    +
    +#endregion
    +
    +using System.Collections.Generic;
    +using System.Linq;
    +using Newtonsoft.Json.Linq;
    +
    +namespace Gremlin.Net.Structure.IO.GraphSON
    +{
    +    internal class Path3Deserializer : IGraphSONDeserializer
    +    {
    +        public dynamic Objectify(JToken graphsonObject, GraphSONReader 
reader)
    +        {
    +            // "labels" is a object[] where each item is ISet<object>
    +            var labelProperty = 
(object[])reader.ToObject(graphsonObject["labels"]);
    +            var labels = labelProperty
    +                .Select(x => new 
HashSet<string>(((ISet<object>)x).Cast<string>()))
    +                .ToList<ISet<string>>();
    +            // "objects" is an object[]
    +            object[] objectsProperty = 
reader.ToObject(graphsonObject["objects"]);
    +            var objects = objectsProperty.ToArray();
    --- End diff --
    
    Isn't `ToArray()` redundant here when `objectsProperty` is already of type 
`object[]`?


> Gremlin .NET support for GraphSON 3.0
> -------------------------------------
>
>                 Key: TINKERPOP-1730
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1730
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: language-variant
>    Affects Versions: 3.3.0
>            Reporter: stephen mallette
>            Priority: Minor
>
> This may be a quite minor thing at this point as [~jorgebg] already had much 
> of this in place. GraphSON 3.0 should be the default for 3.3.0 - right now it 
> is set to 2.0 so all the tests pass.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to