Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #442, xact priority sorted incorrectly #447

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dmknutsen
Copy link
Contributor

Checklist (Please check before submitting)

Describe the contribution
Pulled in fix from @ctolbert555
Marked as draft because I still need to test the update against the open source

Fixed priority sort functionality

Previously, if the priority to be inserted was the lowest number (highest prio) it would have no matches and be inserted into the back of the queue, which is incorrect, as the sort is meant to sort from lowest number to highest. Now, if it is the lowest number it will be inserted into the front of the queue.
Fixes #442

Testing performed
TBD

Expected behavior changes
TBD

System(s) tested on
TBD

Contributor Info - All information REQUIRED for consideration of pull request
Dan Knutsen
NASA Goddard

@dmknutsen dmknutsen self-assigned this May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Transaction priority sorted incorrectly
1 participant