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

    https://github.com/apache/flink/pull/4628#discussion_r140038861
  
    --- Diff: docs/ops/deployment/mesos.md ---
    @@ -226,6 +226,10 @@ When running Flink with Marathon, the whole Flink 
cluster including the job mana
     Takes a comma-separated list of key:value pairs corresponding to the 
attributes exposed by the target
     mesos agents.  Example: `az:eu-west-1a,series:t2`
     
    +`mesos.constraints.soft.balanced`: Soft Constraints for balancing the 
tasks across mesos based on agent attributes (**DEFAULT**: None).
    +Takes a comma-separated list of key=value pairs. Key corresponds to host 
attribute and value is number of expected unique values for given host 
attribute.
    +Example: `az=3,rack_id=4`
    --- End diff --
    
    This syntax seems confusing since it reads like a constraint that `az` will 
be equal to `3`.  How about: `az(3),rack_id(4)`.
    
    Feel free to use a regular expression to easily parse it (just a 
suggestion): `(?:(\w+)\((\d+)\)(?=,|$))`


---

Reply via email to