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

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

Github user fhueske commented on the pull request:

    https://github.com/apache/flink/pull/1928#issuecomment-215527988
  
    Thanks for the update and the explanations @twalthr. 
    I think it would be good to leave comments about the "intentional bugs" 
(missing null check and reference equality) ;-) IntelliJ pointed those out to 
me and others might wonder as well.
    
    Good to merge after the comments are added.


> Add BigDecimal and BigInteger as Basic types
> --------------------------------------------
>
>                 Key: FLINK-3786
>                 URL: https://issues.apache.org/jira/browse/FLINK-3786
>             Project: Flink
>          Issue Type: New Feature
>          Components: Core
>            Reporter: Timo Walther
>            Assignee: Timo Walther
>
> We already had the discussion on the mailing list some months ago about 
> adding BigDecimal and BigInteger as basic types.
> Especially for business or scientific applications it 
> makes sense to support the BigInteger and BigDecimal types natively. In 
> my opinion they are as important as Date or Void and should be added as 
> BasicTypes. The Table API would also benefit from it.
> http://mail-archives.apache.org/mod_mbox/flink-dev/201511.mbox/%3c564cad71.8070...@apache.org%3E



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

Reply via email to