Github user GJL commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6386#discussion_r204368060
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/slotpool/SlotPool.java
 ---
    @@ -1011,7 +1011,7 @@ public void failAllocation(final AllocationID 
allocationID, final Exception caus
                        failPendingRequest(pendingRequest, cause);
                }
                else if (availableSlots.tryRemove(allocationID)) {
    -                   log.debug("Failed available slot [{}] with ", 
allocationID, cause);
    +                   log.debug("Failed available slot with allocation id 
{}.", allocationID, cause);
    --- End diff --
    
    Why remove *allocation id*?


---

Reply via email to