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

    https://github.com/apache/flink/pull/5676#discussion_r175811121
  
    --- Diff: 
flink-core/src/main/java/org/apache/flink/api/common/TaskInfo.java ---
    @@ -107,4 +131,12 @@ public int getAttemptNumber() {
        public String getTaskNameWithSubtasks() {
                return this.taskNameWithSubtasks;
        }
    +
    +   /**
    +    * Returns the allocation id for where this task is executed.
    +    * @return the allocation id for where this task is executed.
    +    */
    +   public String getAllocationID() {
    --- End diff --
    
    If we really want to store the `AllocationID`, then we should store it not 
as a `String` but instead move `AllocationID` to `flink-core`. If this is not 
needed, then I would suggest to rename it something else.


---

Reply via email to