yupeng9 opened a new pull request #6306:
URL: https://github.com/apache/incubator-pinot/pull/6306
## Description
- Add a third parameter to `ST_Point` to create the point in geography,
this simplifies the geographic point creation. Instead of
`ST_GeogFromText(ST_AsText(ST_Point(group_lat, group_lon))`, we can now use
`ST_Point(group_lat, group_lon, 1)`.
- Support `ST_Point` creation from literal such as
`ST_Point(44.183189,-115.761905)`
- Swapped latitude, longitude for `ST_Distance` calculation for geography.
, etc.
## Upgrade Notes
Does this PR otherwise need attention when creating release notes? Things to
consider:
- The signature of `ST_Point` is changed to take an optional third
parameter. 0 (default) means geometry, 1 means geography
- In `ST_Distance(ST_Point( y1,x1), ST_Point(y2, x2))` is changed to
`ST_Distance(ST_Point(x1, y1), ST_Point(x1, y2))` for geography
## Release Notes
The sequence of latitude, longitude is changed for the points in ST_Distance
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
