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 Postgres12, latest ogr2ogr in preload #252

Closed
wants to merge 11 commits into from
Closed

Conversation

nyurik
Copy link
Member

@nyurik nyurik commented May 14, 2020

  • Use PostgreSQL 12
  • skip any features with errors when importing NaturalEarth data
  • Just for the preload image, use the latest ogr2ogr from sid (unstable)

nyurik added 4 commits May 3, 2020 15:33
* Use proper postgis-maintained docker image rather than build our own
* use UTF8PROC 2.5.0
* include postgres gzip support
* Use PostgreSQL 12
* skip any features with errors when importing NaturalEarth data
* Just for the preload image, use the latest ogr2ogr from `sid` (unstable)
@Duiesel
Copy link
Contributor

Duiesel commented May 28, 2020

@nyurik Hi, Can't figure it out in what state is this PR now? Is there any problems with ogr2ogr and PG12? Or it will be merged later?

@Duiesel
Copy link
Contributor

Duiesel commented Jul 1, 2020

@nyurik I've tried ogr2ogr 3.0.4 with -skipfailures and got at least one empty ne_10m_ocean table as a result. So this option is useless in this case. It skipping all layer with failed feature in it I think.
Any updates from @nvkelso ?

UPD: This isn't a bug and option isn't useless, there are only one row in ne_10m_ocean table.

@TomPohys
Copy link
Member

Could we close this when #413 is already merged?

@TomPohys
Copy link
Member

Thanks! Let's close it with regard to #413.

@TomPohys TomPohys closed this May 15, 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