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.

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

fix(db): Fix a sprout/history tree read panic in Zebra 1.4.0, which only happens before the 25.3.0 state upgrade completes #7972

Merged
merged 8 commits into from
Nov 22, 2023
Merged

fix(db): Fix a sprout/history tree read panic in Zebra 1.4.0, which only happens before the 25.3.0 state upgrade completes #7972

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
2116ab7
Avoid a race condition in the 25.3.0 state upgrade
teor2345 Nov 21, 2023
cfd2db8
Look for the most recent sprout tree height if needed
teor2345 Nov 21, 2023
f7a43b7
Get the latest history tree not the tip height history tree
teor2345 Nov 21, 2023
dbd41b7
Discard keys provided by low level database method
teor2345 Nov 21, 2023
78b3b64
Remove extra ;
teor2345 Nov 21, 2023
8aede16
Provide key types & rustfmt
teor2345 Nov 21, 2023
f5a7542
Fix weird closure type syntax
teor2345 Nov 21, 2023
8ff0464
Update zebra-state/src/service/finalized_state/disk_format/upgrade/fi…
teor2345 Nov 22, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix weird closure type syntax
  • Loading branch information
@teor2345
teor2345 committed Nov 21, 2023
commit f5a75428da7d799629e4ebdd4de434d9ca657c65
2 changes: 1 addition & 1 deletion zebra-state/src/service/finalized_state/zebra_db/chain.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ impl ZebraDb {
history_tree = self
.db
.zs_last_key_value(&history_tree_cf)
.map(|(_key: Height, tree_value)| tree_value);
.map(|(_key, tree_value): (Height, _)| tree_value);
}

history_tree.unwrap_or_default()
Expand Down
2 changes: 1 addition & 1 deletion zebra-state/src/service/finalized_state/zebra_db/shielded.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ impl ZebraDb {
sprout_tree = self
.db
.zs_last_key_value(&sprout_tree_cf)
.map(|(_key: Height, tree_value)| tree_value);
.map(|(_key, tree_value): (Height, _)| tree_value);
}

sprout_tree.expect("Sprout note commitment tree must exist if there is a finalized tip")
Expand Down
Loading