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

   ### What problem does this PR solve?
   
   Issue Number: None
   
   Related PR: https://github.com/apache/doris/pull/64627
   
   A nullable runtime-filter result can have no NULLs in the current batch. 
Checking has_null() then treating that result as a bare ColumnUInt8 when the 
probe argument contains NULLs can fail a type assertion and abort the BE. Check 
the physical column type instead.
   
   Master already corrects this path in the COW refactoring #64627. This 
release-branch fix changes only the type check and preserves the existing 
interface.
   
   ### Release note
   
   Fix a possible BE crash when evaluating a null-aware runtime filter.
   
   ### Check List (For Author)
   
   - Test
       - [ ] Regression test
       - [ ] Unit Test
       - [ ] Manual test
   
   No new tests added. Build and tests were not run. clang-format 16 and git 
diff --check passed.
   
   - Behavior changed:
       - [x] Yes. Nullable columns with an all-zero null map use the nullable 
branch.
   
   - Does this need documentation?
       - [x] No.
   
   ### 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