ayubpathan commented on PR #1086: URL: https://github.com/apache/yunikorn-core/pull/1086#issuecomment-4586001015
Thanks @wilfred-s, I checked #1088. The inherited-property reload part of my previous patch overlaps with YUNIKORN-3285, so I removed the ApplyConf merge change and the related inherited-property test from this PR. This PR is now narrowed back to YUNIKORN-3245: resetting runtime fields derived from the effective queue properties in UpdateQueueProperties(), so removed properties revert to their defaults on config update. I kept the mergeProperties cleanup out of this revision to avoid duplicating #1088. If you prefer that cleanup to be handled here instead, I can adjust. -- 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]
