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

Add a dont_clean option #847

Merged
merged 2 commits into from
May 16, 2022
Merged

Add a dont_clean option #847

merged 2 commits into from
May 16, 2022

Conversation

roelarents
Copy link
Contributor

We have a use case with a very large (85+ GiB, 20+ layers) dataset with small features that only need to be served on one zoom level. "Small" means that the features are usually not bigger than 1 tile in the single zoomlevel that they are served on. The features in the dataset are validated beforehand (self intersecting, etc). Clipping, simplifying and cleaning/validating are not necessary in our case, and disabling them increases performance. Especially cleaning takes a lot of processing time, which takes the response time above a workable threshold for this dataset. (Especially when clipping is disabled, but that's a known issue, I believe.)

tl;dr: an option to disable cleaning/making valid

Copy link
Member

@gdey gdey left a comment

Choose a reason for hiding this comment

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

LGTM

@ARolek ARolek merged commit 2113966 into go-spatial:v0.15.x May 16, 2022
@roelarents roelarents deleted the dont-clean branch May 16, 2022 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants