Only One Send or Recevie Flush per Connection Drain

[nibanks]

Description

Updates the connection operation draining logic to only flush send and receive once we call, instead of over and over. The goal here is to improve fairness between connections when they are CPU limited (i.e. both sending or receiving a lot).

Testing

CI/CD

Documentation

N/A

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.02%. Comparing base (a9504fe) to head (319ef1b).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5097      +/-   ##
==========================================
- Coverage   87.36%   86.02%   -1.35%     
==========================================
  Files          59       59              
  Lines       18035    18043       +8     
==========================================
- Hits        15756    15521     -235     
- Misses       2279     2522     +243     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[vgmahajanshetty]

Whether this can have negative impact on this connection throughput? Because I think this operation will be dequeued and processed on next schedule for this connection in worker loop, which might introduce some delay.