Uh oh!
There was an error while loading. Please reload this page.
[Bug](catalog) fix runtime filter partition pruning error with binary type - #59564
Conversation
Thearas
commented
Jan 5, 2026
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
zhangstar333
commented
Jan 5, 2026
run buildall |
hello-stephen
commented
Jan 5, 2026
FE UT Coverage ReportIncrement line coverage |
doris-robot
commented
Jan 5, 2026
TPC-H: Total hot run time: 31832 ms |
doris-robot
commented
Jan 5, 2026
TPC-DS: Total hot run time: 172938 ms |
doris-robot
commented
Jan 5, 2026
ClickBench: Total hot run time: 27.09 s |
zhangstar333
commented
Jan 5, 2026
run buildall |
doris-robot
commented
Jan 5, 2026
TPC-H: Total hot run time: 31389 ms |
doris-robot
commented
Jan 5, 2026
TPC-DS: Total hot run time: 173256 ms |
doris-robot
commented
Jan 5, 2026
ClickBench: Total hot run time: 27.42 s |
doris-robot
commented
Jan 5, 2026
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Jan 6, 2026
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
7223bb9 to
40f886fComparezhangstar333
commented
Jan 6, 2026
run buildall |
doris-robot
commented
Jan 6, 2026
TPC-H: Total hot run time: 31904 ms |
zhangstar333
commented
Jan 6, 2026
run buildall |
doris-robot
commented
Jan 6, 2026
TPC-H: Total hot run time: 31765 ms |
doris-robot
commented
Jan 6, 2026
TPC-DS: Total hot run time: 173439 ms |
doris-robot
commented
Jan 6, 2026
ClickBench: Total hot run time: 26.79 s |
hello-stephen
commented
Jan 6, 2026
FE UT Coverage ReportIncrement line coverage |
zhangstar333
commented
Jan 6, 2026
run buildall |
doris-robot
commented
Jan 6, 2026
TPC-H: Total hot run time: 31814 ms |
doris-robot
commented
Jan 6, 2026
TPC-DS: Total hot run time: 172372 ms |
doris-robot
commented
Jan 6, 2026
ClickBench: Total hot run time: 26.94 s |
hello-stephen
commented
Jan 6, 2026
FE Regression Coverage ReportIncrement line coverage |
hello-stephen
commented
Jan 6, 2026
FE Regression Coverage ReportIncrement line coverage |
1 similar comment
hello-stephen
commented
Jan 6, 2026
FE Regression Coverage ReportIncrement line coverage |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
| return null; | ||
| } | ||
| return value.toString(); | ||
| case BINARY: |
There was a problem hiding this comment.
A corresponding case needs to be added for paimon.
There was a problem hiding this comment.
// binary type as partition key will cause issues in paimon, so skipping these tests
Uh oh!
There was an error while loading. Please reload this page.
… type (apache#59564) ### What problem does this PR solve? Related PR: apache#53399 Problem Summary: the serializePartitionValue function will return String value. But the binary type use String with utf8 will be cause data corrupted, and it is not same with origin data.
… 59473 (#59729) ### What problem does this PR solve? cherry-pick from master #59564 and #59473 ### 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: apache/doris-website#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 -->
… type (apache#59564) ### What problem does this PR solve? Related PR: apache#53399 Problem Summary: the serializePartitionValue function will return String value. But the binary type use String with utf8 will be cause data corrupted, and it is not same with origin data.
… 59473 (apache#59729) ### What problem does this PR solve? cherry-pick from master apache#59564 and apache#59473 ### 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: apache/doris-website#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 -->
What problem does this PR solve?
Related PR: #53399
Problem Summary:
the serializePartitionValue function will return String value.
But the binary type use String with utf8 will be cause data corrupted, and it is not same with origin data.
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)