shuke987 opened a new pull request, #69607:
URL: https://github.com/apache/doris/pull/69607

   ### What problem does this PR solve?
   
   Issue Number: None
   
   Related PR: #69581
   
   Problem Summary: The BE header-dependency gate rejects `util/threadpool.h` 
once its reach exceeds 1087 translation units (the historical baseline of 988 
plus 10%). [Compile build 
1081000](http://43.132.222.7:8111/buildConfiguration/Doris_DorisCompile_Compile/1081000?buildTab=log)
 reported 1088 TUs. Clean master `c3c7af57fa4` now reaches 1089 and reproduces 
the same failure.
   
   Raise this baseline to 1200, making the effective limit 1320 TUs with the 
existing 10% allowance. This is an explicit dependency-budget relaxation to 
unblock builds and allow growth headroom. Other budgets, layering rules and the 
failure behavior remain unchanged; no reduction in compile cost is claimed.
   
   Add an include-injection self-test confirming that spreading `threadpool.h` 
through `common/logging.h` still fails its reverse-reach budget and prints the 
diagnostic command, then restores to a passing state. Related PR #69581 adjusts 
the same entry as part of its Lance feature; this PR isolates the requested 
budget change.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test
       - [ ] Regression test
       - [x] Unit Test
           - `bash build-support/tests/run.sh`: all seven gate self-test groups 
passed, including the new threadpool over-budget injection.
       - [x] Manual test
           - `bash build-support/check-build-hygiene.sh`: fails on clean master 
at 1089 TUs; passes with this patch.
           - `git diff --check`: passed.
           - Local validation used an isolated source snapshot of master 
`c3c7af57fa4`. Full compilation is left to PR CI.
       - [ ] No need to test or manual test.
   
   - Behavior changed:
       - [ ] No.
       - [x] Yes. The build-time threadpool header reach limit rises from 1087 
to 1320 TUs. Doris runtime behavior is unchanged.
   
   - Does this need documentation?
       - [x] No.
       - [ ] Yes.
   
   ### Check List (For Reviewer who merge this PR)
   
   - [ ] Confirm the release note
   - [ ] Confirm test cases
   - [ ] Confirm document
   - [ ] Add branch pick label
   
   


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