zchuango commented on PR #3469: URL: https://github.com/apache/brpc/pull/3469#issuecomment-5364318274
@wwbmmm @chenBright Please carefully review this feature. We have conducted multiple rounds of tests both online and offline, and no abnormal issues have been found. For details, see the following test screenshots and evidence. Additional validation evidence for commit `fc865456fcc21f2e8325e3e9b0c27f7d6fce7b21`. ## Local stress testing - ASan: 200/200 rounds passed. - 6 focused progressive-read timeout tests per round. - 1,200 total test invocations. - 0 failures. - 0 sanitizer errors. - Bazel 8.3.1: 20/20 runs passed. - 120 total focused test invocations. The attached archive contains: - sanitized ASan log; - sanitized Bazel log; - ASan reproduction script; - Bazel reproduction script; - SHA256 checksums for every included file. Archive: [progressive-read-timeout-test-evidence-fc865456.zip](https://github.com/user-attachments/files/31268728/progressive-read-timeout-test-evidence-fc865456.zip) Archive SHA256: `3641592B51D1BCE19F5BA1796FB2F91C27E5980D94CC9E040847D16AF2CFBAA4` ## GitHub Actions reruns Five of six complete Linux workflow attempts passed. Attempt #5 failed only in the existing `clang-unittest-bazel-with-babylon-and-new-pb` job, in `RdmaTest.v3_server_rejects_oversized_pb_size`. The other 10 Linux jobs in that attempt passed. This PR does not modify RDMA code or that test. The subsequent complete attempt #6 passed all 11 Linux jobs, including the previously failing job. - [Attempt #5](https://github.com/BGQ99/brpc/actions/runs/32340822978/attempts/5) - [Attempt #6 — all jobs passed](https://github.com/BGQ99/brpc/actions/runs/32340822978/attempts/6) ## Validation Screenshots <details> <summary>ASan stress test: 200/200 rounds passed</summary> <img width="2411" height="520" alt="asan" src="https://github.com/user-attachments/assets/3b9e7b41-1a99-4edf-91f8-0363979c4e5b"; /> </details> <details> <summary>Bazel 8.3.1 stress test: 20/20 runs passed</summary> <img width="2458" height="432" alt="bazel" src="https://github.com/user-attachments/assets/11fad16f-f82a-4968-9d65-3cbaa33e4a6c"; /> </details> <details> <summary>GitHub Actions attempt #6: all Linux jobs passed</summary> <img width="2487" height="763" alt="CI" src="https://github.com/user-attachments/assets/b17f00ef-ae6b-4820-9979-189c3144590c"; /> </details> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
