Skip to content

Refactor Sender crate#87

Open
Sajjon wants to merge 11 commits intomainfrom
cyon_tps_ng_split_big_fn
Open

Refactor Sender crate#87
Sajjon wants to merge 11 commits intomainfrom
cyon_tps_ng_split_big_fn

Conversation

@Sajjon
Copy link
Contributor

@Sajjon Sajjon commented Sep 11, 2025

Note

Target branch is not main, it is cyon_tps_ng, please review PR #86
before this.

This PR contains a major refactoring of the sender crate, with code clean up and performance adjustments.

Notable changes:

  • Split huge functions into smaller functions and/or methods - by introducing a stateful SenderApp struct
  • Refactor out literal values and replace with consts
  • Send transactions in parallel instead of sequentially, since we were not using the streamed result of submit_and_watch anyway

@Sajjon Sajjon mentioned this pull request Sep 11, 2025
@Sajjon Sajjon changed the title Cyon tps ng split big fn Refactor Sender crate Sep 11, 2025
Base automatically changed from cyon_tps_ng to main September 19, 2025 07:22
@Sajjon
Copy link
Contributor Author

Sajjon commented Sep 19, 2025

We should probably do https://github.com/paritytech/polkadot-stps/tree/cyon_rewrite instead

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.

1 participant