[
https://issues.apache.org/jira/browse/FLINK-4647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15883396#comment-15883396
]
ASF GitHub Bot commented on FLINK-4647:
---------------------------------------
Github user greghogan commented on the issue:
https://github.com/apache/flink/pull/2987
@mushketyk what do you think of refactoring `GraphCsvReader` and
`BipartiteGraphCsvReader` to use configuration objects using the builder
patter? It would be nice to have `CsvReaderConfiguration` in core Flink but we
could add it to Gelly first.
`GraphCsvReader` currently have two of everthing
(fieldDelimiterEdges/fieldDelimiterVertices,
ignoreCommentsEdges/ignoreCommentsVertices, ...). `BipartiteGraphCsvReader`
looks to add three of the same. We could look into collapsing these into a
single configuration object.
> Implement BipartiteGraph reader
> -------------------------------
>
> Key: FLINK-4647
> URL: https://issues.apache.org/jira/browse/FLINK-4647
> Project: Flink
> Issue Type: Sub-task
> Components: Gelly
> Reporter: Ivan Mushketyk
> Assignee: Ivan Mushketyk
>
> Implement reading bipartite graph from a CSV. Should be similar to how
> regular graph is read from a file.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
