JingsongLi commented on issue #9884: [FLINK-14266][table] Introduce 
RowCsvInputFormat to new CSV module
URL: https://github.com/apache/flink/pull/9884#issuecomment-545719375
 
 
   > Thanks @JingsongLi for the PR. I added an initial set of comments. It 
would be great if we could further reduce the number of limitations. The CSV 
format is one of the most important batch connectors and should have a feature 
set similar to the (de)serialization schema. Otherwise we need to document a 
lot of limitations in descriptors and docs.
   
   Thanks @twalthr for your review.
   These limitations are compared with the previous CsvInputFormat in 
flink-java, not the RFC-(de)serialization schema in flink-csv. Some can 
continue to improve, some are more difficult (relying on Jackson).
   You are right, we need to document a lot of limitations in docs.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to