Skip to content

perf: optimize pending queue lookup from O(n) to O(1)#164

Closed
liobrasil wants to merge 1 commit intomainfrom
lionel/optimize-pending-lookup
Closed

perf: optimize pending queue lookup from O(n) to O(1)#164
liobrasil wants to merge 1 commit intomainfrom
lionel/optimize-pending-lookup

Conversation

@liobrasil
Copy link
Copy Markdown
Contributor

Add isPending() function to registry for direct dictionary lookup instead of iterating over array returned by getPendingYieldVaultIDs().

Closes: #???

Description


For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the standards mentioned here.
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

Add isPending() function to registry for direct dictionary lookup
instead of iterating over array returned by getPendingYieldVaultIDs().

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@liobrasil liobrasil requested a review from a team as a code owner February 4, 2026 20:33
@liobrasil
Copy link
Copy Markdown
Contributor Author

Closing as superseded by later scheduler refactors on main; the original hotspot no longer exists on current main.

@liobrasil liobrasil closed this Mar 31, 2026
@liobrasil liobrasil deleted the lionel/optimize-pending-lookup branch March 31, 2026 18:45
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.

3 participants