shuke987 opened a new pull request, #61549:
URL: https://github.com/apache/doris/pull/61549
## Summary
- Replace `#include "runtime/runtime_state.h"` with `class RuntimeState;`
forward declaration in **14 header files**
- These headers only use `RuntimeState*` as pointer/reference parameters —
they never access RuntimeState members or methods
- For files with corresponding `.cpp`, the full `#include` is preserved in
the `.cpp`
This reduces the transitive include fan-out from `runtime_state.h`, which
pulls in heavy protobuf/thrift generated headers. This should improve
incremental build times for files that depend on these 14 headers.
### Modified headers:
`hash_table_set_build.h`, `runtime_filter_producer_helper_{cross,set}.h`,
`tablet_sink_hash_partitioner.h`, `lambda_function.h`,
`short_circuit_evaluation_expr.h`, `vcolumn_ref.h`, `vcondition_expr.h`,
`vectorized_fn_call.h`, `vexpr_context.h`, `vtopn_pred.h`,
`task_execution_context.h`, `collection_statistics.h`, `ann_topn_runtime.h`,
`rowset_reader_context.h`
## Test plan
- [x] Full BE build (1522 targets) passes with zero compilation errors
- [ ] CI pipeline
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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]
