flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-4081) FieldParsers should support empty strings
Date Tue, 02 Aug 2016 14:42:20 GMT

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

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

Github user StephanEwen commented on the issue:

    https://github.com/apache/flink/pull/2297
  
    Using an error state for an empty string seems a bit unorthodox.
    I take it returning `0` does not work for some reason?


> 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)

Mime
View raw message