Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SQL: Add Misc Geometry Property functions #35698

Closed
wants to merge 4 commits into from

Conversation

imotov
Copy link
Contributor

@imotov imotov commented Nov 19, 2018

Adds support for ST_Dimension, ST_GeometryType, ST_X, ST_Y, ST_XMin,
ST_XMax, ST_YMin, ST_YMax functions.

Relates to #29872

Adds support for ST_Dimension, ST_GeometryType, ST_X, ST_Y, ST_XMin,
ST_XMax, ST_YMin, ST_YMax functions.

Relates to elastic#29872
@imotov imotov added >enhancement :Analytics/Geo Indexing, search aggregations of geo points and shapes v7.0.0 :Analytics/SQL SQL querying labels Nov 19, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

@imotov imotov requested review from costin, nknize and astefan November 19, 2018 15:48
@imotov
Copy link
Contributor Author

imotov commented Nov 20, 2018

retest this please

@elasticmachine elasticmachine mentioned this pull request Nov 21, 2018
13 tasks
@imotov imotov added the WIP label Nov 21, 2018
@imotov
Copy link
Contributor Author

imotov commented Nov 21, 2018

Labeled it as WIP for now since I need to do some conflict resolution here first due to #35521.

}

public double minX() {
return shapeBuilder.buildS4J().getBoundingBox().getMinX();
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nknize that's probably not the best way to handle this. Do you have any suggestions on what can be done here?

@imotov imotov removed the WIP label Nov 26, 2018
@jasontedor jasontedor added v8.0.0 and removed v7.0.0 labels Feb 6, 2019
@imotov
Copy link
Contributor Author

imotov commented Apr 11, 2019

Part of this PR is superseded by #41104 and the rest will be implemented on a new basis in a different PR. Closing.

@imotov imotov closed this Apr 11, 2019
@imotov imotov removed the v8.0.0 label Apr 24, 2019
@imotov imotov deleted the geo-sql-add-misc-geometry-funcs branch May 1, 2020 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Geo Indexing, search aggregations of geo points and shapes :Analytics/SQL SQL querying >enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants