Skip to content

avoid using recv in sigverify#9727

Draft
KirillLykov wants to merge 1 commit intoanza-xyz:masterfrom
KirillLykov:klykov/avoid-recv-in-streamer
Draft

avoid using recv in sigverify#9727
KirillLykov wants to merge 1 commit intoanza-xyz:masterfrom
KirillLykov:klykov/avoid-recv-in-streamer

Conversation

@KirillLykov
Copy link
Copy Markdown

@KirillLykov KirillLykov commented Dec 23, 2025

Problem

The current version uses recv_timeout which internally relies on the parking/waiking up mechanism.

This PR is draft because it should land after #9732

Summary of Changes

We use looping with only try_recv instead to reduce synchronization overhead. See profiler output:

@KirillLykov KirillLykov force-pushed the klykov/avoid-recv-in-streamer branch 2 times, most recently from b58cbad to 79f6280 Compare December 23, 2025 15:25
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.7%. Comparing base (f35824c) to head (9acfd12).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #9727   +/-   ##
=======================================
  Coverage    82.7%    82.7%           
=======================================
  Files         854      854           
  Lines      320932   320941    +9     
=======================================
+ Hits       265512   265530   +18     
+ Misses      55420    55411    -9     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@KirillLykov KirillLykov force-pushed the klykov/avoid-recv-in-streamer branch from 79f6280 to 9acfd12 Compare January 27, 2026 20:45
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