yujun777 commented on PR #70400:
URL: https://github.com/apache/doris/pull/70400#issuecomment-6735855403

   <!-- doris-repo-review:v1:begin -->
   ### Local pipeline review — ✅ PASS
   
   ```yaml
   schema: doris-repo-review/v1
   status: PASS
   pr: apache/doris#70400
   commit: f85ac4128339139e79085f0f30129c296e48926d
   base: 5684b974c2bcefbd000a2671fc7b2856f9579240
   reviewed_at: 2026-11-09T16:44+08:00
   reviewer: yujun777
   model: gpt-6-astra
   effort: xhigh
   findings: {blocker: 0, major: 0, minor: 1, nit: 0}
   rounds: 1
   converged: true
   ```
   
   **Notes for maintainers**
   
   - 
`fe/fe-core/src/test/java/org/apache/doris/statistics/cache/StatisticsCacheOverloadTest.java:146`
 — Add a saturation test through the public `StatisticsCache()` constructor: 
all five new overload tests inject their own pools, leaving actual 
queue-capacity and rejection-policy wiring unguarded (F-01, Minor, 
test-coverage; no regression).
   - This was read-only source review, including the actual Caffeine 3.2.4 
implementation. Builds/tests were not rerun, and the default queue capacity of 
100000 was not workload- or heap-benchmarked.
   
   <sub>Reviewed locally with the `doris-repo-review` pipeline. Repository 
policy may accept this receipt for the matching commit; it is not a human 
Apache approval.</sub>
   <!-- doris-repo-review:v1:end -->
   


-- 
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]

Reply via email to