[ 
https://issues.apache.org/jira/browse/FLINK-8316?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16318641#comment-16318641
 ] 

ASF GitHub Bot commented on FLINK-8316:
---------------------------------------

Github user xccui commented on the issue:

    https://github.com/apache/flink/pull/5210
  
    Hi @sunjincheng121, yes, the reported issue should be solved with 
FLINK-8331. In addition, maybe we can add the trailing delimiter as a new 
feature (like the 
[CSVParser.java](https://github.com/apache/commons-csv/blob/master/src/main/java/org/apache/commons/csv/CSVParser.java)
 and 
[CSVFormat.java](https://github.com/apache/commons-csv/blob/master/src/main/java/org/apache/commons/csv/CSVFormat.java#L559)
 in commons-csv). I'll do some refactorings to allow separate trailing 
delimiters for both the input and output. What do you think?


> The CsvTableSink and the CsvInputFormat are not in sync
> -------------------------------------------------------
>
>                 Key: FLINK-8316
>                 URL: https://issues.apache.org/jira/browse/FLINK-8316
>             Project: Flink
>          Issue Type: Bug
>          Components: Table API & SQL
>            Reporter: Xingcan Cui
>            Assignee: Xingcan Cui
>
> As illustrated in [this 
> thread|https://lists.apache.org/thread.html/cfe3b1718a479300dc91d1523be023ef5bc702bd5ad53af4fea5a596@%3Cuser.flink.apache.org%3E],
>  the format for data generated in {{CsvTableSink}} is not compatible with 
> that accepted by {{CsvInputFormat}}. We should unify their trailing 
> delimiters.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to