Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

implement replay binary #1075

Merged
merged 19 commits into from
Nov 21, 2023
Merged

implement replay binary #1075

merged 19 commits into from
Nov 21, 2023

Conversation

juanbono
Copy link
Collaborator

@juanbono juanbono commented Oct 11, 2023 •
edited
Loading

Add a binary to re-execute transactions using RPC

Description

This PR adds several changes:

  • It modifies a bit the RPC StateReader crate to expose the relevant bits needed for using it as a library. I also restructured the crate to be easier to import the SirRPCStateReader and the RPCState structs.
  • It also adds a new crate and binary called replay that uses the RPC to redirect all the calls to State methods and re-execute transactions, blocks and block-ranges. We should work on caching the RPC responses in a future PR.

Checklist

  • Linked to Github Issue
  • Unit tests added
  • Integration tests added.
  • This change requires new documentation.
    • Documentation has been added/updated.

@juanbono juanbono changed the title restructure rpc_state_reader crate implement replay binary Oct 11, 2023
@juanbono juanbono marked this pull request as ready for review November 14, 2023 23:12
fmoletta
fmoletta approved these changes Nov 16, 2023
View reviewed changes
Copy link
Contributor

@fmoletta fmoletta left a comment

Choose a reason for hiding this comment

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

Great! 🚀

pefontana
pefontana approved these changes Nov 16, 2023
View reviewed changes
Copy link
Contributor

@pefontana pefontana left a comment

Choose a reason for hiding this comment

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

LGTM!

@juanbono juanbono added this pull request to the merge queue Nov 21, 2023
Merged via the queue into main with commit 4422734 Nov 21, 2023
7 checks passed
@juanbono juanbono deleted the add_replay_binary branch November 21, 2023 19:06
@juanbono juanbono mentioned this pull request Nov 27, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@fmoletta fmoletta fmoletta approved these changes

@pefontana pefontana pefontana approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@juanbono @pefontana @fmoletta