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

avoid reading legacy db on write #2617

Merged
merged 1 commit into from
May 30, 2021
Merged

avoid reading legacy db on write #2617

merged 1 commit into from
May 30, 2021

Conversation

arnetheduck
Copy link
Member

  • don't consider legacy database when writing state - this read is slow
    on kvstore
  • avoid epoch transition when there's an exact match in cache already
  • simplify init to only consider checkpoint states

* don't consider legacy database when writing state - this read is slow
on kvstore
* avoid epoch transition when there's an exact match in cache already
* simplify init to only consider checkpoint states
@github-actions
Copy link

github-actions bot commented May 30, 2021

Unit Test Results

     28 files  ±0     304 suites  ±0   12m 42s ⏱️ ±0s
   588 tests ±0     570 ✔️ ±0    18 💤 ±0  0 ❌ ±0 
3 228 runs  ±0  3 124 ✔️ ±0  104 💤 ±0  0 ❌ ±0 

Results for commit 60df177. ± Comparison against base commit 60df177.

♻️ This comment has been updated with latest results.

@zah zah merged commit 60df177 into unstable May 30, 2021
@zah zah deleted the write-state-legacy branch May 30, 2021 09:32
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.

2 participants