chenyulin0719 commented on code in PR #426: URL: https://github.com/apache/yunikorn-site/pull/426#discussion_r1588788027
########## docs/user_guide/queue_config.md: ########## @@ -201,7 +201,8 @@ partitions: The placement rules are defined and documented in the [placement rule](placement_rules.md) document. Each partition can have only one set of placement rules defined. -If no rules are defined the placement manager is not started and each application *must* have a queue set on submit. +If no rules are defined, [provided rule](placement_rules#provided-rule) will be applied. +Each application *must* have a queue set on submit. Review Comment: We have another rule 'recoveryRule', for all the placement sets, but it only works for existing applications during initialization. We should keep it hidden from users, so this statement looks good to me. PS: https://github.com/apache/yunikorn-core/blob/3ac7b4856dc88cf02e35ab0b113d949aa0d7438a/pkg/scheduler/placement/placement.go#L199 -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
