Open jackzhhuang opened 2 weeks ago
The pull request introduces several modifications across multiple files. Key changes include the addition of a line to delete all DAG sync blocks in the ensure_dag_parent_blocks_exist
method of the BlockCollector
struct, and updates to the fetch_blocks
method to improve block management. Additionally, the starcoin
StatefulSet configuration is adjusted to remove the sync
directory during startup. The GhostdagManager
struct sees a method rename and changes to error handling, while the PruningPointManagerT
struct receives enhanced debug logging in the next_pruning_point
method.
File Path | Change Summary |
---|---|
sync/src/tasks/block_sync_task.rs | - Modified ensure_dag_parent_blocks_exist to delete all DAG sync blocks under certain conditions. - Updated fetch_blocks to ensure blocks fetched from local store are saved to sync_dag_store . |
kube/manifest/starcoin-vega.yaml | - Updated container args to include removal of the sync directory during startup. |
flexidag/src/ghostdag/protocol.rs | - Renamed _verify_and_ghostdata method and altered its error handling logic. - Removed bail from imports. |
flexidag/src/prune/pruning_point_manager.rs | - Added debug logging in next_pruning_point to output key parameter values for better traceability. |
flexidag/src/blockdag.rs | - Enhanced logging in calc_mergeset_and_tips to clarify details about next_ghostdata . |
flexidag/tests/tests.rs | - Added add_and_print_with_difficulty function for block creation with specified difficulty. - Modified add_and_print_with_pruning_point to standardize difficulty. - Introduced test_verification_blue_block_inconsistent and expanded existing tests for better coverage. |
BlockCollector
struct in block_sync_task.rs
, which is directly related to the changes made in the main PR that also modifies the BlockCollector
struct's methods for managing DAG sync blocks.delete_all_dag_sync_block
in sync_dag_store.rs
, which aligns with the main PR's addition of a line to delete all DAG sync blocks from the sync_dag_store
.block_sync_task.rs
to streamline how absent blocks are processed, which is relevant to the main PR's focus on improving block management during synchronization.find_absent_ancestor
method in block_sync_task.rs
, which is directly related to the changes in the main PR that also involves modifications to methods within the same file.🐇 In the code we hop and play,
Deleting blocks along the way.
With logs so clear, we trace our path,
In sync we dance, avoiding wrath.
A change here, a tweak there,
Together we build, with utmost care! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Pull request type
Please check the type of change your PR introduces:
What is the current behavior?
Issue Number: N/A
What is the new behavior?
-
-
Other information
Summary by CodeRabbit
Release Notes
New Features
Bug Fixes
sync
directory, ensuring a cleaner environment.Documentation
Chores