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

Don't hang in transaction_count #1052

Merged
merged 1 commit into from
Aug 24, 2018

Conversation

sakridge
Copy link
Member

Situation is there can be that there can be bad entries in
the bench-tps CRDT table until they get purged later. Threads however
are created for those bad entries and then will hang on trying
to get the transaction_count from those bad addresses and never end.

@sakridge sakridge requested a review from garious August 24, 2018 02:50
Situation is there can be that there can be bad entries in
the bench-tps CRDT table until they get purged later. Threads however
are created for those bad entries and then will hang on trying
to get the transaction_count from those bad addresses and never end.
@sakridge sakridge force-pushed the no-hang-on-transaction-count branch from 7af8f58 to 39eccd1 Compare August 24, 2018 03:02
@sakridge sakridge merged commit 6fc21a4 into solana-labs:master Aug 24, 2018
@sakridge sakridge deleted the no-hang-on-transaction-count branch August 24, 2018 03:57
vkomenda pushed a commit to vkomenda/solana that referenced this pull request Aug 29, 2021
Bumps [futures](https://github.com/rust-lang/futures-rs) from 0.3.8 to 0.3.9.
- [Release notes](https://github.com/rust-lang/futures-rs/releases)
- [Changelog](https://github.com/rust-lang/futures-rs/blob/master/CHANGELOG.md)
- [Commits](rust-lang/futures-rs@0.3.8...0.3.9)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
brooksprumo added a commit to brooksprumo/solana that referenced this pull request Apr 26, 2024
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.

2 participants