Skip to content

refactor(verifier): improve cache key handling and restore logic#1039

Closed
yu2C wants to merge 1 commit into
mainfrom
vrf-cache
Closed

refactor(verifier): improve cache key handling and restore logic#1039
yu2C wants to merge 1 commit into
mainfrom
vrf-cache

Conversation

@yu2C

@yu2C yu2C commented Jul 7, 2026

Copy link
Copy Markdown
Contributor
  • Removed the clearing of the verifier cache during state restoration, as it may no longer be necessary.
  • Introduced a new structure for the ring verifier cache key that includes both epoch and gammaK hash, enhancing cache key uniqueness.
  • Updated the cache access logic to utilize the new key structure, ensuring that the verifier cache is maintained correctly across state restorations.
  • Added tests to verify that the verifier cache remains intact after restoring state, confirming the expected behavior of the cache under different conditions.
image runs avg wall time
before b23010a 35.46s / 31.83s / 36.85s 34.71s
after cache-key change 31.71s / 30.22s / 34.64s 32.19s

- Removed the clearing of the verifier cache during state restoration, as it may no longer be necessary.
- Introduced a new structure for the ring verifier cache key that includes both epoch and gammaK hash, enhancing cache key uniqueness.
- Updated the cache access logic to utilize the new key structure, ensuring that the verifier cache is maintained correctly across state restorations.
- Added tests to verify that the verifier cache remains intact after restoring state, confirming the expected behavior of the cache under different conditions.
@yu2C yu2C requested a review from YCC3741 July 7, 2026 15:19
@yu2C yu2C self-assigned this Jul 7, 2026
@yu2C yu2C added the refactor refactoring production code, eg. renaming a variable label Jul 7, 2026
@yu2C yu2C closed this Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor refactoring production code, eg. renaming a variable

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant