[feature](function) Add ST_IsClosed function - #67350
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
|
/review |
TPC-H: Total hot run time: 16807 ms |
TPC-DS: Total hot run time: 82164 ms |
ClickBench: Total hot run time: 14.76 s |
FE UT Coverage ReportIncrement line coverage |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
|
@morrySnow Hi, all TeamCity checks for this PR have passed. The required I tried Thanks! |
|
/review |
|
@zclllyybb Hi, this PR implements All TeamCity checks have passed, and the related documentation PR (#4102) is linked. The required Could you please help trigger the automated review or take a look when convenient? Thanks! |
|
/review |
|
Codex automated review failed and did not complete. Error: Your access token could not be refreshed because your refresh token was already used. Please log out and sign in again. Please inspect the workflow logs and rerun the review after the underlying issue is resolved. |
What problem does this PR solve?
Issue Number: ref #48203
Related PR: apache/doris-website#4102
Problem Summary:
Add the
ST_IsClosedspatial scalar function.For a valid LineString, the function returns:
truewhen the first and last points are exactly equalfalsewhen the LineString is openNULLfor aNULLinputNULLfor a non-LineString or invalid encoded geometryThe implementation uses exact S2 point equality without introducing a tolerance.
This PR also adds Nereids registration and visitor support, FE and BE unit tests, and a self-asserting SQL regression suite.
Release note
Add the
ST_IsClosedspatial function.Check List (For Author)
Test
Regression test
Unit Test
VGeoFunctionsTest.*passedManual test
No need to test or manual test
Behavior changed:
Does this need documentation?