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

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

Github user greghogan commented on the issue:

    https://github.com/apache/flink/pull/2469
  
    Do you have a second use case in mind for adding this function to 
`MathUtils`? My thought would be to keep this separate to avoid confusion 
between signed and unsigned downcasts.


> Convert to negative in LongValueToIntValue
> ------------------------------------------
>
>                 Key: FLINK-4572
>                 URL: https://issues.apache.org/jira/browse/FLINK-4572
>             Project: Flink
>          Issue Type: Bug
>          Components: Gelly
>    Affects Versions: 1.2.0
>            Reporter: Greg Hogan
>            Assignee: Greg Hogan
>            Priority: Minor
>
> The Gelly drivers expect that scale 32 edges, represented by the lower 32 
> bits of {{long}} values, can be converted to {{int}} values. Values between 
> 2^31 and 2^32 - 1 should be converted to negative integers, which is not 
> supported by {{MathUtils.checkedDownCast}}.



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

Reply via email to