Mryange opened a new pull request, #65327:
URL: https://github.com/apache/doris/pull/65327
### What problem does this PR solve?
Nested loop join may receive a single probe or build block whose row count
is larger than the runtime batch size. Some probe-side generation loops checked
whether the next appended rows would still fit in `state->batch_size()` before
entering the loop. When the input block itself was oversized, those loops
skipped the body entirely and did not advance probe/build progress.
Root cause: the batch-size guard was used as a loop entry condition, so
oversized but otherwise valid blocks prevented the operator from processing at
least one chunk.
This change converts the affected nested loop join generation loops to `do
while`, so each call can make progress even when one input block is larger than
batch size. It also removes the hard debug check that required
`_join_block.rows()` to stay within batch size, because oversized input can now
legitimately produce an oversized output block for that processing step.
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should
merge into -->
--
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]
