PoiBlackTea opened a new pull request, #1075:
URL: https://github.com/apache/yunikorn-k8shim/pull/1075
### Description
During scheduler recovery, `Context.registerPods()` sorts recovered pods
with the intention of making queue ordering consistent across restarts.
However, because `CreationTimestamp` has 1-second resolution at source and the
informer list order is non-deterministic (Go map iteration), sorting on
`CreationTimestamp` alone produces arbitrary ordering when multiple pods share
the exact same second. The same non-deterministic ordering occurs during
per-application task dispatch in `Application.getTasks()`.
This PR fixes the issue by using `UID` (`taskID`) as a deterministic
secondary tie-breaker when `CreationTimestamp` values are identical.
### Type of change
Please delete options that are not relevant.
- [X] Bug Fix
- [ ] Improvement
- [ ] Feature
- [ ] Refactoring
- [ ] Documentation
### Jira issue
Jira ID : https://issues.apache.org/jira/browse/YUNIKORN-3354
- [ ] I have created a Jira issue for this pull request.
- [X] The Jira ID is part of the title of this pull request.
### AI Tooling
If an AI tool was used:
- [X] The PR includes the phrase "Generated by Antigravity CLI", where
Antigravity CLI is the name of the AI tool used.
- [X] My use of AI contributions follows the ASF legal policy.
Check https://www.apache.org/legal/generative-tooling.html for details.
### How has this been tested?
- [ ] New unit tests were added to cover new or changed code paths.
- [X] `make test_all` was run, and no failures reported.
- [ ] `make e2e_test` was run, and no failures reported.
- [ ] new e2e tests have been added.
### Questions:
- [ ] The change needs documentation, a pull request for
apache/yunikorn-site repository will be created.
- [ ] There is breaking changes for older versions: jira is tagged with
`release-notes` label.
- [ ] The licenses files needs to be updated.
### Screenshots or other details
N/A
--
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]
