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

ASF GitHub Bot commented on FLINK-4624:
---------------------------------------

GitHub user s1ck opened a pull request:

    https://github.com/apache/flink/pull/2527

    [FLINK-4624] Allow for null values in Graph Summarization

    * Bug was caused by serializers that cannot handle null values (e.g. Long)
    * VertexGroupItem now uses Either<NullValue, VV> instead of VV
    * Generalized test cases
    * Added tests for vertex/edge values of type Long

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

    $ git pull https://github.com/s1ck/flink FLINK-4624

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

    https://github.com/apache/flink/pull/2527.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 #2527
    
----
commit e6db894d6b84cf95206905a6f1a6713f78e32988
Author: Martin Junghanns <martin.jungha...@gmx.net>
Date:   2016-09-21T11:31:41Z

    [FLINK-4624] Support null values in Graph Summarization
    
    * Bug was caused by serializers that cannot handle null values (e.g. Long)
    * VertexGroupItem now uses Either<NullValue, VV> instead of VV
    * Generalized test cases
    * Added tests for vertex/edge values of type Long

----


> Gelly's summarization algorithm cannot deal with null vertex group values
> -------------------------------------------------------------------------
>
>                 Key: FLINK-4624
>                 URL: https://issues.apache.org/jira/browse/FLINK-4624
>             Project: Flink
>          Issue Type: Bug
>          Components: Gelly
>            Reporter: Till Rohrmann
>            Assignee: Martin Junghanns
>             Fix For: 1.2.0
>
>
> Gelly's {{Summarization}} algorithm cannot handle null values in the 
> `VertexGroupItem.f2`. This behaviour is hidden by using Strings as a vertex 
> value in the {{SummarizationITCase}}, because the {{StringSerializer}} can 
> handle null values. 



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

Reply via email to