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

Performance Improvements, pt. II #3577

Closed
arboleya opened this issue Jan 11, 2025 · 2 comments
Closed

Performance Improvements, pt. II #3577

arboleya opened this issue Jan 11, 2025 · 2 comments
Assignees
Labels
feat Issue is a feature

Comments

@arboleya
Copy link
Member

Building upon part one (#3576) of this series, and while we strive for faster transaction processing time, we should now dissect everything from start to end regarding all network requests necessary for the entire transaction flow and cycle—but now from a browser perspective.

Unlike in part one, this chapter must go beyond the submission and analyze everything that happens until the end, including transaction settlement and (optional) retrieval. It should not be limited to the SDK but include everything from a connectors/wallet perspective, which will inevitably shift the focus to front-end applications.

We should be creative and consider optimistic techniques and whatever else we can.

The end goal is to make tx's super snappy.

cc @LuizAsFight

@danielbate
Copy link
Member

We have now completely dissected the end to end transaction process, the full report can be found here.

For the end to end transaction flow, improvements can be summarised as follows:

  • Transactions are pre-loaded (if possible) and estimated and funded through an improved dry-run endpoint, as outlined in Performance Improvements, pt. I #3576.
  • Data transmission between the SDK and Wallet is improved through changes to the JSON RPC interface to give the wallet more context of actions that have already been performed by the SDK, to reduce redundancy and network calls being performed by the wallet
  • Wallet level transaction simulation after signing is optional, to negate unnecessary simulation
  • Tx submission is owned by the SDK, to reduce the number of calls required to build a transaction summary

Follow up issues will soon follow and be posted in this issue, but will then supersede this issue.

@danielbate
Copy link
Member

This issue is being superseded by the issues that were the outcome of this investigation:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Issue is a feature
Projects
None yet
Development

No branches or pull requests

2 participants