This repository has been archived by the owner on Jan 22, 2025. It is now read-only.
Workaround linux hang #137
Merged
+2
−2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Without this patch, Linux systems would hang when running the demo. The root cause (why Linux is acting differently than macOS) was not determined, but we know the problem is caused by a known issue in the transaction pipeline - that entries are not pulled off the historian channel until after the full transaction batch is processed. This patch makes the sync_channel large enough that it should never block on a gigabit network.
|
@rlkelly, fyi |
Codecov Report
@@ Coverage Diff @@
## master #137 +/- ##
=======================================
Coverage 96.39% 96.39%
=======================================
Files 18 18
Lines 1886 1886
=======================================
Hits 1818 1818
Misses 68 68
Continue to review full report at Codecov.
|
steviez
added a commit
to steviez/solana
that referenced
this pull request
Mar 8, 2024
…abs#137) ReplayStage owning the pool allows for subsequent work to configure the size of the pool; configuring the size of the pool inside of the lazy_static would have been a little messy
willhickey
pushed a commit
that referenced
this pull request
Mar 9, 2024
ReplayStage owning the pool allows for subsequent work to configure the size of the pool; configuring the size of the pool inside of the lazy_static would have been a little messy
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Without this patch, Linux systems would hang when running the demo.
The root cause (why Linux is acting differently than macOS) was
not determined, but we know the problem is caused by a known
issue in the transaction pipeline - that entries are not pulled
off the historian channel until after the full transaction batch
is processed. This patch makes the sync_channel large enough that
it should never block on a gigabit network.