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

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

Github user twalthr commented on the issue:

    https://github.com/apache/flink/pull/2303
  
    @fhueske I think we should keep the methods as they currently are. I 
renamed them to "nextNumericString"/"nextNumericStringEndPos" and added a 
explanation what a numeric string is. You are right they don't do much numeric 
at the moment but only the numeric classes will use it. Furthermore, the error 
states and exceptions are numeric. E.g. if we don't do the whitespace checking 
in `nextNumericString` we also need to return the position instead of the 
string.


> CsvTableSource does not support reading SqlTimeTypeInfo types
> -------------------------------------------------------------
>
>                 Key: FLINK-4248
>                 URL: https://issues.apache.org/jira/browse/FLINK-4248
>             Project: Flink
>          Issue Type: Improvement
>          Components: Table API & SQL
>    Affects Versions: 1.1.0
>            Reporter: Till Rohrmann
>            Assignee: Timo Walther
>
> The Table API's {{CsvTableSource}} does not support to read all Table API 
> supported data types. For example, it is not possible to read 
> {{SqlTimeTypeInfo}} types via the {{CsvTableSource}}.



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

Reply via email to