I agree it's a bit disjoint. The validation for group sizes is being done 
[here](https://github.com/apache/aurora/pull/37/files/49646d0097546cc070fb0a4b3555cca482dceb28#diff-2b88151c5903ffbb593ec7812574f9c4R812)
 and I couldn't come up with a scenario where this wouldn't protect us from a 
bad update config.

Do you think I should abstract that code to a function and call it here as well?


I thought it was 

[ Full content available at: https://github.com/apache/aurora/pull/37 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to