search

panjf2000 / gnet

🚀 gnet is a high-performance, lightweight, non-blocking, event-driven networking framework written in pure Go.
https://gnet.host
Apache License 2.0
9.7k stars 1.04k forks source link

opt: mitigate the latency issue by prioritizing asynchronous writes #563

Closed panjf2000 closed 8 months ago

panjf2000 commented 8 months ago

By prioritizing asynchronous writes.

Fixes #423

codecov[bot] commented 8 months ago

Codecov Report

Attention: Patch coverage is 81.25000% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 79.08%. Comparing base (ab69eec) to head (efb0d94). Report is 2 commits behind head on dev.

Files Patch % Lines
engine_unix.go 60.00% 2 Missing :warning:
acceptor_unix.go 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #563 +/- ## ======================================= Coverage 79.08% 79.08% ======================================= Files 25 25 Lines 2109 2109 ======================================= Hits 1668 1668 Misses 305 305 Partials 136 136 ``` | [Flag](https://app.codecov.io/gh/panjf2000/gnet/pull/563/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Andy+Pan) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/panjf2000/gnet/pull/563/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Andy+Pan) | `79.08% <81.25%> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Andy+Pan#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

panjf2000 commented 8 months ago

See also #218, this might bring that old issue back.