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

Upgrade to PostgreSQL 14 #413

Merged
merged 5 commits into from
Aug 2, 2022
Merged

Upgrade to PostgreSQL 14 #413

merged 5 commits into from
Aug 2, 2022

Conversation

TomPohys
Copy link
Member

@TomPohys TomPohys commented Jul 29, 2022

This PR is upgrading PostgreSQL from 9.6 to 14.

GEOS 3.9.0 is used in Debian-bullseye image for building postgis:14-3.2. I kept the code for the eventual manual update of GEOS.

There is also tune-up PostgreSQL to turn off jit which prolongs the import-sql step in OpenMapTiles as #252.

Copy link
Collaborator

@lazaa32 lazaa32 left a comment

Choose a reason for hiding this comment

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

Typo in Docker Automated build - missing d

@TomPohys TomPohys merged commit 36e31f7 into master Aug 2, 2022
@TomPohys TomPohys deleted the PostgreSQL14 branch August 2, 2022 09:57
TomPohys added a commit to openmaptiles/openmaptiles that referenced this pull request Aug 2, 2022
This PR updating Natural Earth dataset from v4.1 to 5.1.2. It is in cooperation with openmaptiles/openmaptiles-tools#414, where is `import-data` docker image updated.

PR removes manual fixes for lake merging (already fixed in version 5.1.2)

Switched comparison from `gn_ascii` (removed from NE5) to `name_en`.

Could be merged after openmaptiles/openmaptiles-tools#413. After OMT-T is merged, it will use `latest` image (until the release of OMT-T v7)
pathmapper added a commit to pathmapper/openmaptiles-tools that referenced this pull request Aug 8, 2022
Comment not needed anymore after openmaptiles#413
TomPohys pushed a commit that referenced this pull request Aug 8, 2022
Comment not needed anymore after #413
@jsanz jsanz mentioned this pull request Sep 28, 2022
@nyurik
Copy link
Member

nyurik commented Sep 28, 2022

@TomPohys , I'm very happy PG got updated to v14, but please, when you copy code from another PR, please link to it and credit the author. Just to keep track of what comes from where.

@TomPohys
Copy link
Member Author

Hi @nyurik, I am afraid, that I do not exactly remember how I wrote the code since it was in July. I am sorry if it looks like I took some code from your PR. It wasn't my intention. I will be more careful in the future. I edit the description.

The main motivation was to not show the warning for users during generating tiles about jit (and of course speed up import-sql step)

lambda v: 'disable JIT in PG 11+ for complex queries' if v != 'off' else '',

And the style template is just a few lines below.

echo "++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++"
echo " Loading OMT postgis extensions"
echo "++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++"
for db in template_postgis "$POSTGRES_DB"; do
echo "Loading extensions into $db"
PGUSER="$POSTGRES_USER" "${psql[@]}" --dbname="$db" <<-'EOSQL'

francois2metz pushed a commit to indoorequal/openmaptiles-tools that referenced this pull request Mar 17, 2023
This PR is upgrading PostgreSQL from 9.6 to 14.

GEOS 3.9.0 is used in Debian-bullseye image for building [postgis:14-3.2](https://github.com/postgis/docker-postgis/blob/master/14-3.2/Dockerfile#L1). I kept the code for the eventual manual update of GEOS.

There is also tune-up PostgreSQL to turn off `jit` which prolongs the `import-sql` step in OpenMapTiles.
francois2metz pushed a commit to indoorequal/openmaptiles-tools that referenced this pull request Mar 17, 2023
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