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

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

Github user twalthr commented on the issue:

    https://github.com/apache/flink/pull/2297
  
    The length is not sufficient to differentiate between empty column and 
empty string.
    E.g. if you have quoted strings: `,,` and `,"",`
    With the changes in this PR we don't modify the return values, but deposit 
the information of an empty column in the error state.


> FieldParsers should support empty strings
> -----------------------------------------
>
>                 Key: FLINK-4081
>                 URL: https://issues.apache.org/jira/browse/FLINK-4081
>             Project: Flink
>          Issue Type: Bug
>          Components: Core
>            Reporter: Flavio Pompermaier
>            Assignee: Timo Walther
>              Labels: csvparser, table-api
>
> In order to parse CSV files using the new Table API that converts rows to Row 
> objects (that support null values), FiledParser implementations should 
> support emptry strings setting the parser state to 
> ParseErrorState.EMPTY_STRING (for example FloatParser and DoubleParser 
> doesn't respect this constraint)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to