Github user greghogan commented on the pull request:
https://github.com/apache/flink/pull/1900#issuecomment-214825734
By reuse I was referring to needing `MapVertexValueLongToString` and
`MapEdgeValueLongToString` since the current `Graph` methods operate at the
level of `Vertex` and `Edge`. And then to translate labels you need
`MapVertexLabelLongToString` and `MapEdgeLabelLongToString`. In this PR we
simply have a single `LongToString` which can be used to translate vertex
labels, edge labels, vertex values, and edge values.
I agree that `Translate` methods taking a `Graph` as input would be better
as methods on `Graph`.
And if we don't implement as `MapFunction` then we still need `Translate`
to handle the `TypeInformation`.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
