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

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

GitHub user spmallette opened a pull request:

    https://github.com/apache/incubator-tinkerpop/pull/321

    TINKERPOP-1308 Allow Serialization to "reference"

    https://issues.apache.org/jira/browse/TINKERPOP-1308
    
    Added a "lite" serializer option for gryo in Gremlin Server. This 
serializer will write graph elements to "referenced" rather than "detached". 
The default serialization behavior for Gremlin Server has not changed as a 
result of this work.  Using the "lite" serialization is not the default, but an 
option to be made on the request.
    
    Everything is good with:
    
    ```text
    mvn clean install && mvn verify -pl gremlin-server 
-DskipIntegrationTests=false
    ```
    
    VOTE +1

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/incubator-tinkerpop TINKERPOP-1308

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-tinkerpop/pull/321.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 #321
    
----
commit 2e030502804116edc4fb636acf4f5fbe3ae3e2ce
Author: Stephen Mallette <[email protected]>
Date:   2016-05-20T12:33:59Z

    Allow calls to addCustom on GryoMapper to override.
    
    By allowing override, users get complete control over the gryo 
serialization process and open up more options for Gremlin Server to offer 
other optional serialization views over gryo.

commit f811a8aecbfa82b1747503a88b13af71b23e3ae9
Author: Stephen Mallette <[email protected]>
Date:   2016-05-20T16:43:12Z

    Added gryo "lite" serializer.
    
    Serializes graph elements to "reference" rather than "detached". Altered 
all the configuration files that required the change. This serializer is just 
an option - the default configuration from the client side is to use the 
existing serializer.

commit ace64ded32539ae7377068bfb6c481a46f7e7200
Author: Stephen Mallette <[email protected]>
Date:   2016-05-20T17:09:02Z

    Renamed gryo method for better consistency with graphson.

commit 6aa3e98264b8633862b7e9dd646278258bc4a42e
Author: Stephen Mallette <[email protected]>
Date:   2016-05-20T17:23:32Z

    Clean up scoping of configuration methods for gryo message serializer.

commit 6d3f18276fbe7803d775779ad6832ef6a6174639
Author: Stephen Mallette <[email protected]>
Date:   2016-05-23T20:37:54Z

    Refactored gryo serialization tests to have better re-use.

----


> Serialize to "reference" for Gremlin Server
> -------------------------------------------
>
>                 Key: TINKERPOP-1308
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1308
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: driver, io, server
>    Affects Versions: 3.2.0-incubating
>            Reporter: stephen mallette
>            Assignee: stephen mallette
>             Fix For: 3.2.1
>
>
> Gremlin Server currently returns "detached" graph elements. This approach can 
> be expensive for a {{Vertex}} as it means that the graph has to get all the 
> properties and serialize them - especially true for graphs with 
> multi-properties. 
> This issue is to provide an option for Gremlin Server to return "reference" 
> graph elements which only return the id of the element. The default option 
> will continue to be "detached" so as not to introduce a breaking change. We 
> can consider at a later date whether or not to switch the default.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to