Github user shghatge commented on the pull request:
https://github.com/apache/flink/pull/847#issuecomment-123057981
The only problem with assuming NullValue if a value is missing is that we
can't return NullValue in place of VV.
I mean to say Graph<K, VV, EV> in this VV or EV can't be NullValue.
otherwise that was what I was originally going for.
Maybe since any of the other methods to create DataSet/Graph don't provide
a method to give EdgeValue as NullValue and just expect the user to map it (at
least that is what I saw), maybe we could just remove the functionality. I had
only added it since many examples seemed to use it so I thought it would be
nice to have that functionality.
In any case we can just keep one typesNullEdge method too because if they
don't want that, they can use normal overloaded types, 3 arguments for no
NullValue, 2 arguments for null vertex and 1 argument for null vertex and edge
and just one method named typesNullEdge to tell that only edges have NullValue.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
