vtlim commented on code in PR #12555: URL: https://github.com/apache/druid/pull/12555#discussion_r921711072
########## docs/development/geo.md: ########## @@ -78,25 +108,36 @@ The grammar for a spatial filter is as follows: } ``` +The order of the dimension coordinates in the spatial filter must be equal to the order of the dimension coordinates in the `spatialDimensions` array. + ### Bound types -#### `rectangular` +The `bound` property of the spatial filter object lets you filter on ranges of dimension values. +You can define rectangular, radius, and polygon filter bounds. + +#### Rectangular + +The `rectangular` bound has the following elements: + +|Property|Description|Required| +|--------|-----------|--------| +|`minCoords`|The list of minimum dimension coordinates in the form [latitude, longitude]|yes| +|`maxCoords`|The list of maximum dimension coordinates in the form [latitude, longitude]|yes| + +#### Radius -|property|description|required?| -|--------|-----------|---------| -|minCoords|List of minimum dimension coordinates for coordinates [x, y, z, …]|yes| -|maxCoords|List of maximum dimension coordinates for coordinates [x, y, z, …]|yes| +The `radius` bound has the following elements: -#### `radius` +|Property|Description|Required| +|--------|-----------|--------| +|`coords`|Origin coordinates in the form [x, y]|yes| +|`radius`|The float radius value|yes| -|property|description|required?| -|--------|-----------|---------| -|coords|Origin coordinates in the form [x, y, z, …]|yes| -|radius|The float radius value|yes| +#### Polygon -#### `polygon` +The `polygon` bound has the following elements: -|property|description|required?| -|--------|-----------|---------| -|abscissa|Horizontal coordinate for corners of the polygon|yes| -|ordinate|Vertical coordinate for corners of the polygon|yes| +|Property|Description|Required| +|--------|-----------|--------| +|`abscissa`|Horizontal coordinate for the corners of the polygon|yes| +|`ordinate`|Vertical coordinate for the corners of the polygon|yes| Review Comment: ```suggestion |`abscissa`|Horizontal coordinates for the corners of the polygon|yes| |`ordinate`|Vertical coordinates for the corners of the polygon|yes| ``` minimum length is 3: https://github.com/apache/druid/blob/master/processing/src/main/java/org/apache/druid/collections/spatial/search/PolygonBound.java#L82-L85 -- 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]
