[GitHub] flink pull request #4660: [FLINK-7050][table] Add support of RFC compliant C...

[uybhatti]

Github user uybhatti commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4660#discussion_r141336157
  
    --- Diff: flink-libraries/flink-table/pom.xml ---
    @@ -114,6 +114,12 @@ under the License.
                        <groupId>joda-time</groupId>
                        <artifactId>joda-time</artifactId>
                </dependency>
    +           <!-- FOR RFC 4180 Compliant CSV Parser -->
    +           <dependency>
    --- End diff --
    
    I think it's better to move this functionality into `flink-connectors` as  
`flink-connector-csv`, what do you think?


---