xintongsong commented on a change in pull request #11615:
URL: https://github.com/apache/flink/pull/11615#discussion_r411319138



##########
File path: docs/_includes/generated/expert_scheduling_section.html
##########
@@ -14,6 +14,12 @@
             <td>Boolean</td>
             <td>Enable the slot spread out allocation strategy. This strategy 
tries to spread out the slots evenly across all available <span 
markdown="span">`TaskExecutors`</span>.</td>
         </tr>
+        <tr>
+            <td><h5>slotmanager.number-of-slots.max</h5></td>

Review comment:
       Given that there's another ticket (FLINK-15959) for introducing min 
slots limit to Flink cluster, I think it might be good to have a pair of 
`xyz.[min|max]`. Of course one could also argue for a pair of `min-xyz` 
and`max-xyz`. I've not strong preference here.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to