Yukang-Lian opened a new pull request, #63816:
URL: https://github.com/apache/doris/pull/63816
### What problem does this PR solve?
Issue Number: None
Related PR: #62263, #61647
Problem Summary: This PR picks two approved changes into `branch-4.0`:
- #62263: use segment footer `raw_data_bytes` for first-time vertical
compaction batch-size estimation, with focused BE unit coverage.
- #61647: avoid delete quorum stalls when delete push tasks fail before
enough successful replicas return, with FE unit coverage.
### Release note
None
### Check List (For Author)
- Test:
- `PATH="/mnt/disk2/lianyukang/ldb_toolchain_v16/bin:$PATH"
./build-support/clang-format.sh`
- `PATH="/mnt/disk2/lianyukang/ldb_toolchain_v16/bin:$PATH"
./build-support/check-format.sh`
- `./run-fe-ut.sh --run
org.apache.doris.load.DeleteJobTest,org.apache.doris.master.MasterImplDeleteTaskTest`
- `./run-be-ut.sh --run
--filter=VerticalCompactionTest.TestFirstCompactionUsesFooterEstimation:VerticalCompactionTest.TestFooterRawDataBytesAccuracy:VerticalCompactionTest.TestFooterRawDataBytesNullableSparse`
- `build-support/run-clang-tidy.sh` is not present on this `branch-4.0`
checkout. I attempted manual `clang-tidy -p be/ut_build_ASAN` on the touched
C++ files; the unfiltered run failed on existing branch-wide tidy findings and
a current v26 toolchain `stddef.h` lookup error, so it was not a usable
changed-line signal.
- Behavior changed: Yes. This backports the behavior from #62263 and #61647
to `branch-4.0`.
- Does this need documentation: No
--
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]
