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

    https://github.com/apache/flink/pull/5671#discussion_r174139638
  
    --- Diff: 
flink-clients/src/main/java/org/apache/flink/client/program/rest/RestClusterClient.java
 ---
    @@ -567,7 +567,7 @@ public GetClusterStatusResponse getClusterStatus() {
     
        @Override
        public int getMaxSlots() {
    -           return 0;
    +           return -1;
    --- End diff --
    
    I suggest to assign this magic number to some `static final` variable with 
a descriptive name and use it here and in `CliFrontend`, so that the connection 
is clear.


---

Reply via email to