Github user greghogan commented on the issue:
https://github.com/apache/flink/pull/2527
Will merge today. I'll add a note that the `TypeInformation` will be
removed once we resolve the issues typing `Either`.
---
If your project is set up for it, you can reply to this email and have
Github user s1ck commented on the issue:
https://github.com/apache/flink/pull/2527
@greghogan since I addressed all your comments, is there anything left to
do that prevents this from merging?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user s1ck commented on the issue:
https://github.com/apache/flink/pull/2527
> I'd also like to add a ToNullValue translator that would accept any type
and convert to NullValue.
I don't know if I get this right. Do you mean an additional test case with
`NullValue` as
Github user s1ck commented on the issue:
https://github.com/apache/flink/pull/2527
> Would a graph translator simplify the conversion from Long to String? You
can do graph.run(new TranslateEdgeValues<...>(new StringToLong()) and write a
simple public class StringToLong implements
Github user s1ck commented on the issue:
https://github.com/apache/flink/pull/2527
> I'm not following why specifying the TypeInformation is now required with
the change to using Either. Is the type system failing to handle this properly?
You are right, I was experimenting with
Github user greghogan commented on the issue:
https://github.com/apache/flink/pull/2527
I'm not following why specifying the `TypeInformation` is now required with
the change to using `Either`. Is the type system failing to handle this
properly?
---
If your project is set up for
Github user greghogan commented on the issue:
https://github.com/apache/flink/pull/2527
I'd also like to add a `ToNullValue` translator that would accept any type
and convert to `NullValue`.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user greghogan commented on the issue:
https://github.com/apache/flink/pull/2527
Would a graph translator simplify the conversion from Long to String? You
can do `graph.run(new TranslateEdgeValues<...>(new StringToLong())` and write a
simple `public class StringToLong