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

Update condition of index planet_osm_polygon_military #3682

Merged
merged 1 commit into from
Feb 15, 2019

Conversation

Nakaner
Copy link
Contributor

@Nakaner Nakaner commented Feb 14, 2019

This reflects changes to the SQL query of layer landuser-overlay introduced by commit 52085d3 (pull request #3469). This partial index was introduced for this layer and its WHERE condition needs to be updated as well if the WHERE condition of the SQL query used by that layer is changed.

There are no visual changes but rendering performance of this layer should become better.

@kocio-pl kocio-pl added the code label Feb 14, 2019
@kocio-pl kocio-pl added this to the Bugs and improvements milestone Feb 14, 2019
@kocio-pl kocio-pl removed the code label Feb 14, 2019
@kocio-pl kocio-pl removed this from the Bugs and improvements milestone Feb 14, 2019
This reflects changes to the SQL query of layer landuser-overlay
introduced by commit 52085d3 (pull
request gravitystorm#3469).
@matthijsmelissen matthijsmelissen merged commit 5e7c9dd into gravitystorm:master Feb 15, 2019
@matthijsmelissen
Copy link
Collaborator

Thanks!

@matthijsmelissen
Copy link
Collaborator

@kocio-pl In the next release, we should recommend users of this style to re-create their indexes (or at least the planet_osm_polygon_military index) using the provided style.

@kocio-pl
Copy link
Collaborator

I see, could you take care of this (or maybe the whole next release)?

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