Uh oh!
There was an error while loading. Please reload this page.
[opt](function)Some geo functions incorrectly used get. - #40107
Conversation
doris-robot
commented
Aug 29, 2024
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
Mryange
commented
Aug 29, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Aug 29, 2024
TPC-H: Total hot run time: 38078 ms |
doris-robot
commented
Aug 29, 2024
TPC-DS: Total hot run time: 192621 ms |
doris-robot
commented
Aug 29, 2024
ClickBench: Total hot run time: 31.64 s |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
Mryange
commented
Sep 3, 2024
run buildall |
doris-robot
commented
Sep 3, 2024
TPC-H: Total hot run time: 38703 ms |
doris-robot
commented
Sep 3, 2024
TPC-DS: Total hot run time: 193068 ms |
doris-robot
commented
Sep 3, 2024
ClickBench: Total hot run time: 32.54 s |
Mryange
commented
Sep 3, 2024
run buildall |
doris-robot
commented
Sep 3, 2024
TeamCity be ut coverage result: |
doris-robot
commented
Sep 3, 2024
TPC-H: Total hot run time: 37937 ms |
doris-robot
commented
Sep 3, 2024
TPC-DS: Total hot run time: 192161 ms |
doris-robot
commented
Sep 3, 2024
ClickBench: Total hot run time: 32.38 s |
## Proposed changes ``` mysql [test]>select count(st_distance_sphere(db, db, db, db)) from double_ranges; +-------------------------------------------+ | count(st_distance_sphere(db, db, db, db)) | +-------------------------------------------+ | 0 | +-------------------------------------------+ 1 row in set (1.25 sec) mysql [test]>select count(st_distance_sphere(db, db, db, db)) from double_ranges; +-------------------------------------------+ | count(st_distance_sphere(db, db, db, db)) | +-------------------------------------------+ | 0 | +-------------------------------------------+ 1 row in set (0.33 sec) ``` <!--Describe your changes.-->
…Sphere (#45508) ### What problem does this PR solve? Issue Number: close #xxx Related PR: #40107 Problem Summary: In #40107 we introduced a wrong behaviour which would lead to BE crash because of dangling reference. now fix it. ### Release note fix BE crash of function StDistanceSphere and StAngleSphere ### Check List (For Author) - Test <!-- At least one of them must be included. --> - [x] 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: - [x] No. - [ ] Yes. <!-- Explain the behavior change --> - Does this need documentation? - [x] 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 -->
…Sphere (apache#45508) ### What problem does this PR solve? Issue Number: close #xxx Related PR: apache#40107 Problem Summary: In apache#40107 we introduced a wrong behaviour which would lead to BE crash because of dangling reference. now fix it. ### Release note fix BE crash of function StDistanceSphere and StAngleSphere ### Check List (For Author) - Test <!-- At least one of them must be included. --> - [x] 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: - [x] No. - [ ] Yes. <!-- Explain the behavior change --> - Does this need documentation? - [x] 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 -->
## Proposed changes ``` mysql [test]>select count(st_distance_sphere(db, db, db, db)) from double_ranges; +-------------------------------------------+ | count(st_distance_sphere(db, db, db, db)) | +-------------------------------------------+ | 0 | +-------------------------------------------+ 1 row in set (1.25 sec) mysql [test]>select count(st_distance_sphere(db, db, db, db)) from double_ranges; +-------------------------------------------+ | count(st_distance_sphere(db, db, db, db)) | +-------------------------------------------+ | 0 | +-------------------------------------------+ 1 row in set (0.33 sec) ``` <!--Describe your changes.-->
…Sphere (apache#45508) ### What problem does this PR solve? Issue Number: close #xxx Related PR: apache#40107 Problem Summary: In apache#40107 we introduced a wrong behaviour which would lead to BE crash because of dangling reference. now fix it. ### Release note fix BE crash of function StDistanceSphere and StAngleSphere ### Check List (For Author) - Test <!-- At least one of them must be included. --> - [x] 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: - [x] No. - [ ] Yes. <!-- Explain the behavior change --> - Does this need documentation? - [x] 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 -->
Proposed changes