-
Notifications
You must be signed in to change notification settings - Fork 632
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
feat: State Sync from External Storage #8789
Merged
near-bulldozer
merged 89 commits into
near:master
from
nikurt:nikurt-state-syncing-from-s3
Apr 14, 2023
Merged
Changes from all commits
Commits
Show all changes
89 commits
Select commit
Hold shift + click to select a range
0039c20
feat: Dump state to S3
nikurt e9cb32c
fix
nikurt 95ff6dc
Metrics
nikurt c3a0112
Metrics
nikurt 85a8ded
Metrics
nikurt 4f0d69c
Metrics
nikurt 556248f
Add config options, change state part file naming - include num_parts.
nikurt b15dd00
Add config options, change state part file naming - include num_parts.
nikurt 3e17aba
Add config options, change state part file naming - include num_parts.
nikurt 8105005
Refactor state syncing to avoid very long functions.
nikurt cb2731e
Syncing works
nikurt 98cd7cf
Metrics
nikurt 712ca55
Fix
nikurt bc953ea
Fields are not optional
nikurt 61671f5
.
nikurt 0667d8b
Throttling of requests now works! :pray:
nikurt 0063fa8
Too much output
nikurt f9b7f24
Timeouts
nikurt 2195391
Use colour for state sync phase display
nikurt 8b1f311
Fix DB corruption
nikurt b224412
feat: use split store in the view client (#8656)
wacban 5846e1a
feat: Dump state of every epoch to S3 (#8661)
nikurt 4cb5139
feat: Dump state to S3
nikurt afcce1b
Merge
nikurt 8408ce7
Fix: Rework the state-parts tool API
nikurt c61ff2a
Merge
nikurt b682424
Print state_root
nikurt 4f43f8c
Print state_root
nikurt 161a357
Changelog
nikurt 020d7f9
Merge
nikurt 4910fcc
Merge
nikurt c2fa4a0
Merge
nikurt 14e202f
Handle partial state parts dumps
nikurt 0da5f3b
More output
nikurt 610e820
More output
nikurt 296dda3
Fix off-by-one error in the state-part tool.
nikurt 11e965a
fmt
nikurt 95c1ad5
fmt
nikurt dc89915
fmt
nikurt f14e8aa
read-state-header
nikurt 15d1fbb
Dump correct state root in state dump
nikurt f390b19
Merge
nikurt 89134c5
Don't retry S3 errors until the timeout
nikurt 5a1acff
Tune verbosity
nikurt 4a9aa86
Timing
nikurt 56dc868
S3 downloads should now work.
nikurt 74773e0
Metrics for part application.
nikurt 81d511f
Metrics for part application.
nikurt 6db2638
Metrics for part application.
nikurt 51dbd55
More metrics for part download
nikurt 6022323
Ready for review
nikurt cca8bdc
Rename config field
nikurt b800bdc
Fix retry metric
nikurt 1cd3a72
Minimize
nikurt b319c51
fix: apply `log_summary_style` to state sync phase message (#8735)
nikurt 6b09e0d
[Runtime Epoch Split] (3/n) Add ability to get Arc<EpochManagerAdapte…
robin-near fdaa79e
feat: Disable state sync by default because it's unreliable (#8730)
nikurt 89297d4
Merge
nikurt 699954f
Merge
nikurt 1244ac3
Merge
nikurt ad7ded8
Merge
nikurt 6691bf7
Doc
nikurt 1cc6f05
Fix test
nikurt 0ad29cd
Fix test
nikurt 438e343
Move some code to #8794
nikurt ea0b82a
chore: update changelog with 1.32.1 and 1.32.2 (#8869)
wacban e3baa8e
feat: override NetworkConfig from JSON config (#8871)
VanBarbascu 5a0a4fb
feat: Dump state to S3
nikurt 526225d
Fix DB corruption
nikurt f5d9545
Merge
nikurt cc49285
Changelog
nikurt 8067add
More output
nikurt 4c53eac
Merge
nikurt cd34b4a
Merge
nikurt 8f2c9b4
.
nikurt faa8a10
Merge
nikurt cfce87c
Introduce StateSyncInner instead of StateSyncMode
nikurt e71a0ea
merge
nikurt b01050f
merge
nikurt b61749a
Re-fixed. Persist `new_shard_sync` when S3 download completes.
nikurt 594cafa
.
nikurt a9debc5
Remove unneeded enum value and simplify target sampling.
nikurt 00f70d4
Remove `unwrap()`s when creating an S3 bucket.
nikurt 0855002
Merge
nikurt b09200c
Merge
nikurt 27423e5
Merge
Ekleog-NEAR 2a7454f
re-push
nikurt b29d2b3
fix
nikurt 6500c42
Merge branch 'master' into nikurt-state-syncing-from-s3
nikurt File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's why
Clone
was implemented by hand for this struct though right? Where therun_me
field is explicitly reallocated and copied. So could just do the same in theClone
impl for the response field too. Don't have an opinion really on which way is more clear, but could consider doing that