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

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

Github user StephanEwen commented on the issue:

    https://github.com/apache/flink/pull/5205
  
    I think you can test the `MemoryManager` by simply setting pre-allocation 
to false and calling the method to compute the memory size.
    
    For the other two, not sure how to best do that...


> Missing cast in integer arithmetic in 
> TransactionalIdsGenerator#generateIdsToAbort
> ----------------------------------------------------------------------------------
>
>                 Key: FLINK-8037
>                 URL: https://issues.apache.org/jira/browse/FLINK-8037
>             Project: Flink
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Greg Hogan
>            Priority: Minor
>
> {code}
>   public Set<String> generateIdsToAbort() {
>     Set<String> idsToAbort = new HashSet<>();
>     for (int i = 0; i < safeScaleDownFactor; i++) {
>       idsToAbort.addAll(generateIdsToUse(i * poolSize * 
> totalNumberOfSubtasks));
> {code}
> The operands are integers where generateIdsToUse() expects long parameter.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to