manirajv06 commented on PR #830: URL: https://github.com/apache/yunikorn-core/pull/830#issuecomment-2090089967
> -1. I know that there is a spreadsheet of behavioral cases that has been created for this to document how the preemption logic should behave in a variety of circumstances. I would absolutely like to see a PR consisting only of test cases _for each of the identified scenarios_. We can disable the failing ones for now, but that would give us a baseline for how "compliant" the existing preemption logic is. Then YUNIKORN-2500 can be rebased on top of that, re-enabling the tests that it fixes. That will ensure we don't regress compared to the existing behavior. PR https://github.com/apache/yunikorn-core/pull/847 covers all the different test cases but skipped for now. Once the same has been merged, this pr will fix all those test cases with necessary changes. -- 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]
