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

Add note to book about drone throughput limitations #7953

Merged
merged 3 commits into from
Jan 24, 2020

Conversation

jstarry
Copy link
Contributor

@jstarry jstarry commented Jan 24, 2020

Problem

The drone section of the book describes an abstraction which would allow millions of transactions to be processed in a few seconds but makes no mention of the workarounds needed to achieve that goal given Solana's runtime limitations.

Summary of Changes

Add a note to inform readers that a single payer account can only be used once per tick and recommending to have a drone service manage many accounts to increase throughput.

Fixes #

@jstarry jstarry requested a review from garious January 24, 2020 01:01
garious
garious previously approved these changes Jan 24, 2020
Copy link
Contributor

@garious garious left a comment

Choose a reason for hiding this comment

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

just the one nit

@@ -16,6 +16,8 @@ Creator of on-chain game tic-tac-toe hosts a drone that responds to airdrop requ

Creator of a new on-chain token \(ERC-20 interface\), may wish to do a worldwide airdrop to distribute its tokens to millions of users over just a few seconds. That drone cannot spend resources interacting with the Solana cluster. Instead, the drone should only verify the client is unique and human, and then return the signature. It may also want to listen to the Solana cluster for recent entry IDs to support client retries and to ensure the airdrop is targeting the desired cluster.

Note: the Solana cluster will not parallelize transactions funded by the same fee payer account. This means that the throughput of a single fee payer account is limited to the number of _ticks_ processed per second by the current leader. In order to circumvent this limitation, it's advised to have a drone manage many accounts for signing and funding transactions.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you replace the last sentence with something more positive (and not in the passive tense)? How about: "Add additional fee-paying accounts to improve throughput."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@jstarry jstarry added the automerge Merge this Pull Request automatically once CI passes label Jan 24, 2020
@mergify mergify bot dismissed garious’s stale review January 24, 2020 02:47

Pull request has been modified.

@solana-grimes solana-grimes merged commit 3c17db4 into solana-labs:master Jan 24, 2020
@jstarry jstarry deleted the drone-throughput branch January 24, 2020 02:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants