jacktengg commented on PR #68281:
URL: https://github.com/apache/doris/pull/68281#issuecomment-6058354882

   <!-- doris-repo-review:v1:begin -->
   ### Local pipeline review — ✅ PASS
   
   ```yaml
   schema: doris-repo-review/v1
   status: PASS
   pr: apache/doris#68281
   commit: 823a65e337fa45d3408646da2335f3ec2e234542
   base: c91757aed0b44ffbd23c9f0859c428c2033f95ba
   reviewed_at: 2026-10-08T18:57+08:00
   reviewer: jacktengg
   model: gpt-6-astra
   effort: xhigh
   findings: {blocker: 0, major: 0, minor: 0, nit: 0}
   rounds: 1
   converged: true
   ```
   
   **Notes for maintainers**
   
   - `fe/fe-core/src/main/java/org/apache/doris/qe/SessionVariable.java:5870` — 
Prior F-01 is fixed: historical concurrency values normalize before hint origin 
capture, and both SQL/HTTP cleanup paths clear bookkeeping in finally.
   - Full PR review covers all 15 changed files. Existing LOAD/routine-load, 
HTTP load_stream_per_node, global-session journal replay and mixed-version BE 
concerns remain tracked by prior review threads; duplicate exclusion does not 
declare them fixed.
   - Read-only review; author logs corroborate 39 passing FE tests, a 
successful FE build with Checkstyle, and generated/rerun session-concurrency 
regression output. The review team did not execute builds or tests.
   
   <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