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

    https://github.com/apache/flink/pull/5593#discussion_r171181582
  
    --- Diff: 
flink-yarn/src/main/java/org/apache/flink/yarn/AbstractYarnClusterDescriptor.java
 ---
    @@ -321,6 +323,14 @@ public void setZookeeperNamespace(String 
zookeeperNamespace) {
                this.zookeeperNamespace = zookeeperNamespace;
        }
     
    +   public String getNodeLabel() {
    +           return nodeLabel;
    +   }
    +
    +   public void setNodeLabel(String nodeLabel) {
    +           this.nodeLabel = nodeLabel;
    +   }
    --- End diff --
    
    @tillrohrmann the setter for `nodelabel` I followed the example of 
`setZookeeperNamespace` and `setQueue`, it seems that the 
`AbstractYarnClusterDescriptor` constructor can not set this field if we do not 
change it. 


---

Reply via email to