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 bug: no store the sync dag block #4259

Open
wants to merge 16 commits into
base: dag-master
Choose a base branch
from
Open

Fix bug: no store the sync dag block #4259

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
a6403bc
no store the sync dag block
jackzhhuang Nov 4, 2024
c173559
add rm sync in kube file
jackzhhuang Nov 4, 2024
48c9bac
use ghostdata to verify the dag block
jackzhhuang Nov 6, 2024
c8b259a
use ghodata if the verify is not correct
jackzhhuang Nov 6, 2024
1a6c851
use ghostdata instead of verify
jackzhhuang Nov 6, 2024
264a5d6
add verification code for discovering the blue blocks error
jackzhhuang Nov 6, 2024
3b2a145
verify and if it fails it use ghostdata
jackzhhuang Nov 7, 2024
197bc4f
bail out if the ghost data checking is wrong
jackzhhuang Nov 7, 2024
adfcb23
remove to print red blocks avoiding lots log data
jackzhhuang Nov 8, 2024
e06797d
test dag verification
jackzhhuang Nov 11, 2024
86ee296
add log to trace
jackzhhuang Nov 11, 2024
8250159
add reachability data with parents
jackzhhuang Nov 11, 2024
573bc89
fix fmt
jackzhhuang Nov 13, 2024
e3dca70
use unordered_mergeset_without_selected_parent when storing into the …
jackzhhuang Nov 13, 2024
248b7fb
rm commented codes
jackzhhuang Nov 14, 2024
e4eff6c
add uncle test in reachability
jackzhhuang Nov 14, 2024
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
2 changes: 1 addition & 1 deletion sync/src/tasks/block_sync_task.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -454,6 +454,7 @@ where
if block_header.number() % ASYNC_BLOCK_COUNT == 0
|| block_header.number() >= self.target.target_id.number()
{
self.sync_dag_store.delete_all_dag_sync_block()?;
self.find_absent_ancestor(vec![block_header.clone()])
.await?;

Expand All @@ -474,7 +475,6 @@ where
block: block.clone(),
children: vec![],
})?;
self.sync_dag_store.save_block(block)?;
anyhow::Ok(ParallelSign::NeedMoreBlocks)
}
};
Expand Down
Loading