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

Nominatim 4.1.0 #361

Merged
merged 11 commits into from
Aug 24, 2022
Merged

Nominatim 4.1.0 #361

merged 11 commits into from
Aug 24, 2022

Conversation

philipkozeny
Copy link
Collaborator

@philipkozeny philipkozeny commented Aug 23, 2022

  • updated nominatim to 4.1
  • updated base image to 22.04
  • therefore also switched to PostgreSQL 14
  • removed the tokenizer hack
  • optimised graceful shutdowns
  • removed 3.7 from currently supported versions
  • added nominatim admin warm command after initial import

@philipkozeny philipkozeny changed the title WIP: add nominatim 4.1.0 Nominatim 4.1.0 Aug 23, 2022
@philipkozeny
Copy link
Collaborator Author

@leonardehrenfried I think I am done with all the required changes and cleanup. The tests are working, so everything should be fine. Maybe you have time for a quick review, thanks!

Copy link
Collaborator

@leonardehrenfried leonardehrenfried left a comment

Choose a reason for hiding this comment

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

This looks like solid work. Thanks!

philipkozeny and others added 2 commits August 24, 2022 10:07
Co-authored-by: Leonard Ehrenfried <mail@leonard.io>
@philipkozeny philipkozeny merged commit a4c4f05 into mediagis:master Aug 24, 2022
@philipkozeny philipkozeny deleted the 4.1.0 branch August 24, 2022 10:20
@leonardehrenfried
Copy link
Collaborator

I think the combined commit message of the merge to master contained "[skip ci]" so that's why the CI isn't running there.

@philipkozeny
Copy link
Collaborator Author

Triggered it with a small readme change

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.

2 participants