[
https://issues.apache.org/jira/browse/FLINK-3856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15270778#comment-15270778
]
ASF GitHub Bot commented on FLINK-3856:
---------------------------------------
Github user fhueske commented on the pull request:
https://github.com/apache/flink/pull/1959#issuecomment-216893377
Just a few minor comments. What do you think about moving the types to a
`SqlTimeTypeInfo` class, @twalthr? @StephanEwen would that be OK for you?
> Create types for java.sql.Date/Time/Timestamp
> ---------------------------------------------
>
> Key: FLINK-3856
> URL: https://issues.apache.org/jira/browse/FLINK-3856
> Project: Flink
> Issue Type: New Feature
> Components: Core
> Reporter: Timo Walther
> Assignee: Timo Walther
>
> At the moment there is only the {{Date}} type which is not sufficient for
> most use cases about time.
> The Table API would also benefit from having different types as output result.
> I would propose to add the three {{java.sql.}} types either as {{BasicTypes}}
> or in an additional class {{TimeTypes}}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
