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

    https://github.com/apache/flink/pull/2463#discussion_r77523985
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/taskexecutor/TaskExecutorGateway.java
 ---
    @@ -32,4 +33,11 @@
        // 
------------------------------------------------------------------------
     
        void notifyOfNewResourceManagerLeader(String address, UUID 
resourceManagerLeaderId);
    +
    +   /**
    +    * Send by the ResourceManager to the TaskExecutor
    +    * @param allocationID id for the request
    +    * @param resourceManagerLeaderID current leader id of the 
ResourceManager
    +    */
    +   void requestSlot(AllocationID allocationID, UUID 
resourceManagerLeaderID);
    --- End diff --
    
    How is the confirmation of the `TaskExecutor` sent back to the 
`SlotManager`? Would it make sense to send it back via the return value of this 
method?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to