MiYuyuyuyu opened a new pull request, #66814:
URL: https://github.com/apache/doris/pull/66814
### What problem does this PR solve?
Issue Number: close #48203
Related PR: #xxx
Problem Summary:
Add four Trino compatible bounding box accessor functions:
st_xmax / st_xmin / st_ymax / st_ymin.
BE:
- Add GeoShape::bounding_box() (BoundingBox struct) and implement it for
GeoPoint, GeoLine, GeoPolygon, GeoMultiPolygon and GeoCircle (treated as
its center point).
- Add the four function structs in functions_geo.cpp, decoding the input
geometry via GeoShape::from_encoded and returning NULL for invalid input.
FE:
- Add the four Nereids scalar function signatures and register them in
BuiltinScalarFunctions and ScalarFunctionVisitor.
Tests:
- Regression test suite nereids_scalar_fn_st_bounding_box (14 cases,
auto-generated .out).
- BE unit tests GeoTypesTest.bounding_box_* (28 tests passed).
### Release note
Add st_xmax / st_xmin / st_ymax / st_ymin functions.
### Check List (For Author)
- Test
- [x] Regression test
- [x] Unit Test
- Behavior changed:
- [x] Yes. (New SQL functions)
- Does this need documentation?
- [x] Yes. https://github.com/apache/doris-website/pull/4065
--
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]
