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.

Sign up for GitHub

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

Jump to bottom

RED-259: add cache for nodelistByStates #303

Open

dnlbui wants to merge 3 commits into red-103 from RED-259/nodeListFromStates-cache

+16 −0

Contributor

dnlbui commented Sep 18, 2024

https://linear.app/shm/issue/RED-259/add-cache-for-nodelistbystatesnodelistfromstates

Has been rebased on RED-103. SHOULD BE MERGED AFTER RED-103

Summary: Implement Caching for nodelistFromStates Function

caching mechanism for the nodelistFromStates function to improve performance.
nodelistCache Map to store cached results.
Cycle-based cache invalidation using lastCachedInCycle.
tested w/ 20+5 network + rotation + shm-transfer tests

dnlbui added 3 commits

September 18, 2024 16:25


          add nodelist caching for improved performance


          add counters and use correct currentCycle

2a6031b


          remove verbose typing + counters

1f0d12f

Contributor Author

dnlbui commented Sep 19, 2024

please let me know if I should add counters or logs here. I removed ones I added in latest commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet