fix(sidecar): prefer poll() for channel#1443
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1443 +/- ##
==========================================
- Coverage 71.31% 71.30% -0.01%
==========================================
Files 413 413
Lines 66155 66157 +2
==========================================
- Hits 47176 47172 -4
- Misses 18979 18985 +6
🚀 New features to boost your workflow:
|
BenchmarksComparisonBenchmark execution time: 2026-01-13 01:21:58 Comparing candidate commit b63861c in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 57 metrics, 2 unstable metrics. CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
Group 14
Group 15
Group 16
Group 17
Group 18
Group 19
BaselineOmitted due to size. |
This fixes paninc()'ing on a fd number higher than FD_SETSIZE. Signed-off-by: Bob Weinand <[email protected]>
431fef2 to
b63861c
Compare
morrisonlevi
left a comment
There was a problem hiding this comment.
How does using poll fix the issue? I think a comment is warranted here.
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-apple-darwin
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-apple-darwin
x86_64-unknown-linux-gnu
|
|
|
/merge |
|
View all feedbacks in Devflow UI.
This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
The expected merge time in
|
This fixes paninc()'ing on a fd number higher than FD_SETSIZE.