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

[8.16] Try to simplify geometries that fail with TopologyException (#115834) #115842

Merged
merged 2 commits into from
Oct 30, 2024

Conversation

iverase
Copy link
Contributor

@iverase iverase commented Oct 29, 2024

Backports the following commits to 8.16:

…115834)

This geometries are valid and they can actually be simplified so lets make the clipping algorithm a best effort and 
return the original geometry in those cases so the simplification can handle it.
@iverase iverase added :Analytics/Geo Indexing, search aggregations of geo points and shapes >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Oct 29, 2024
@iverase iverase reopened this Oct 30, 2024
@iverase
Copy link
Contributor Author

iverase commented Oct 30, 2024

@elasticmachine update branch

@elasticsearchmachine elasticsearchmachine merged commit 94158e1 into elastic:8.16 Oct 30, 2024
15 checks passed
@iverase iverase deleted the backport/8.16/pr-115834 branch October 30, 2024 06:45
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 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.16.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants