[
https://issues.apache.org/jira/browse/FLINK-1512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14377067#comment-14377067
]
ASF GitHub Bot commented on FLINK-1512:
---------------------------------------
Github user chiwanpark commented on the pull request:
https://github.com/apache/flink/pull/426#issuecomment-85285193
@fhueske Hi, Thanks for your kindly advice! I will fix them soon.
About the order of POJO fields, I think also that the option 3 is good.
However, [FLINK-1665](https://issues.apache.org/jira/browse/FLINK-1665) is not
implemented yet. I would implement the option 1 and 2 now. After FLINK-1665
completed, we can implement the option 3.
About the inheritance of `ScalaCsvInputFormat`, I didn't think about Record
API. Your opinion looks good. I will revert the changes and refactor
`GenericCsvInputFormat` to contain duplicated methods.
> Add CsvReader for reading into POJOs.
> -------------------------------------
>
> Key: FLINK-1512
> URL: https://issues.apache.org/jira/browse/FLINK-1512
> Project: Flink
> Issue Type: New Feature
> Components: Java API, Scala API
> Reporter: Robert Metzger
> Assignee: Chiwan Park
> Priority: Minor
> Labels: starter
>
> Currently, the {{CsvReader}} supports only TupleXX types.
> It would be nice if users were also able to read into POJOs.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
