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

refactor: nuking L1 block number from L2 block #5275

Conversation

benesjan
Copy link
Contributor

Fixes #5274

Copy link
Contributor Author

benesjan commented Mar 18, 2024


// create the block number -> block hash mapping to ensure we retrieve the appropriate events
const blockNumberToBodyHash: { [key: number]: Buffer | undefined } = {};
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This map was completely unused here.

@@ -209,50 +210,50 @@ export class Archiver implements ArchiveSource {
);

const blockBodies = retrievedBlockBodies.retrievedData.map(([blockBody]) => blockBody);

await this.store.addBlockBodies(blockBodies);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code bellow here was all messy so I cleaned that up.

@benesjan benesjan requested a review from LHerskind March 18, 2024 10:02
@benesjan benesjan marked this pull request as ready for review March 18, 2024 10:02
@benesjan benesjan force-pushed the 03-17-refactor_nuking_l1_to_l2_messages_from_block_body branch from d8b572c to 5d407f1 Compare March 18, 2024 10:45
@benesjan benesjan force-pushed the 03-18-refactor_nuking_L1_block_number_from_L2_block branch from 1348fdc to 268d641 Compare March 18, 2024 10:46
@benesjan benesjan force-pushed the 03-17-refactor_nuking_l1_to_l2_messages_from_block_body branch from 5d407f1 to 5c58881 Compare March 18, 2024 11:11
@benesjan benesjan force-pushed the 03-18-refactor_nuking_L1_block_number_from_L2_block branch from 268d641 to 8754701 Compare March 18, 2024 11:12
@benesjan benesjan force-pushed the 03-17-refactor_nuking_l1_to_l2_messages_from_block_body branch from 5c58881 to e34dec8 Compare March 18, 2024 11:28
@benesjan benesjan force-pushed the 03-18-refactor_nuking_L1_block_number_from_L2_block branch from 8754701 to 422191a Compare March 18, 2024 11:28
@benesjan benesjan force-pushed the 03-17-refactor_nuking_l1_to_l2_messages_from_block_body branch from e34dec8 to a1abb66 Compare March 18, 2024 11:56
@benesjan benesjan force-pushed the 03-18-refactor_nuking_L1_block_number_from_L2_block branch from 422191a to 2da0404 Compare March 18, 2024 11:56
@benesjan benesjan force-pushed the 03-17-refactor_nuking_l1_to_l2_messages_from_block_body branch from a1abb66 to a57663a Compare March 18, 2024 12:37
@benesjan benesjan force-pushed the 03-18-refactor_nuking_L1_block_number_from_L2_block branch from 2da0404 to fee8bd3 Compare March 18, 2024 12:37
@benesjan benesjan changed the base branch from 03-17-refactor_nuking_l1_to_l2_messages_from_block_body to master March 18, 2024 12:38
@benesjan benesjan enabled auto-merge (squash) March 18, 2024 12:39
@benesjan benesjan force-pushed the 03-18-refactor_nuking_L1_block_number_from_L2_block branch from fee8bd3 to b33b3d8 Compare March 18, 2024 12:51
@benesjan benesjan changed the base branch from master to 03-17-refactor_l1_l2_messages_cleanup March 18, 2024 12:51
@benesjan benesjan merged commit 6e69a6b into 03-17-refactor_l1_l2_messages_cleanup Mar 18, 2024
8 of 25 checks passed
@benesjan benesjan deleted the 03-18-refactor_nuking_L1_block_number_from_L2_block branch March 18, 2024 12:52
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.

refactor(Misc): Remove l1 block number from L2Block
2 participants