-
Couldn't load subscription status.
- Fork 179
fix: BlockBufferService forceful BN Switch #21803
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Derek Riley <[email protected]>
Signed-off-by: Derek Riley <[email protected]>
Signed-off-by: Derek Riley <[email protected]>
Signed-off-by: Derek Riley <[email protected]>
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Codecov Report❌ Patch coverage is
@@ Coverage Diff @@
## main #21803 +/- ##
============================================
- Coverage 71.81% 71.57% -0.25%
+ Complexity 24549 24487 -62
============================================
Files 2670 2675 +5
Lines 103887 103963 +76
Branches 10864 10881 +17
============================================
- Hits 74608 74410 -198
- Misses 25235 25489 +254
- Partials 4044 4064 +20
... and 143 files with indirect coverage changes 🚀 New features to boost your workflow:
|
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferences |
Signed-off-by: Derek Riley <[email protected]>
Signed-off-by: Derek Riley <[email protected]>
Signed-off-by: Derek Riley <[email protected]>
Description:
This pull request refactors the block node connection management logic to improve how connections are cleaned up and rescheduled, especially during stream resets and forced switches. The changes remove redundant cleanup methods, introduce a configurable delay for rescheduling connections after a forced switch, and update related tests to reflect the new behavior. The code now exposes connection state for easier management and testing.
Connection Cleanup and Rescheduling Improvements
connectionResetsTheStreamandremoveConnectionAndClearActivemethods fromBlockNodeConnectionManager, consolidating connection cleanup logic and making rescheduling more explicit. Stream resets now directly trigger selection of a new block node for streaming.BlockNodeConnectionForced Switch Handling
forcedSwitchRescheduleDelaytoBlockNodeConnectionConfig, allowing the delay for rescheduling a closed active connection after a forced switch to be set via configuration.Testing
Related issue(s):
Fixes #21734
Notes for reviewer:
Checklist