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` i
Github user mushketyk commented on the issue:
https://github.com/apache/flink/pull/2987
Hi @greghogan ,
I've rebased this PR on top of master.
---
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
Github user greghogan commented on the issue:
https://github.com/apache/flink/pull/2987
#2832 has been committed to master. Please rebase.
---
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
Github user greghogan commented on the issue:
https://github.com/apache/flink/pull/2987
There are some refactorings in #2832 which overlap with updates here.
---
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
