BiteTheDDDDt opened a new pull request, #63456:
URL: https://github.com/apache/doris/pull/63456
### What problem does this PR solve?
Issue Number: None
Related PR: None
Problem Summary: Partitioned hash join spill recovery could continue normal
repartition progress after cancellation because some loops stopped on
`state->is_cancelled()` but then fell through to completion handling. This
could mark partially recovered or repartitioned spill data as complete. This PR
returns the cancellation status before advancing partition state, clears
recovered build data during close, and replaces a debug-only child EOS
assertion with a runtime error.
### Release note
None
### Check List (For Author)
- Test: Manual test
- BE object compile: `DORIS_HOME=$PWD ninja -C be/ut_build_ASAN
src/exec/CMakeFiles/Exec.dir/operator/partitioned_hash_join_probe_operator.cpp.o`
- Format check: `build-support/clang-format.sh`;
`build-support/check-format.sh`; `git diff --check`
- Static analysis: `build-support/run-clang-tidy.sh --build-dir
be/ut_build_ASAN --files
be/src/exec/operator/partitioned_hash_join_probe_operator.cpp` was attempted,
but this environment reports pre-existing `be/src/util/jni-util.h`
`static_assert(false)` clang-diagnostic errors unrelated to this change.
- Behavior changed: No
- 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]
