Opened 3 years ago

Closed 21 months ago

#5075 closed defect (fixed)

postgis_restore.pl: ERROR: function "box2d_out" already exists with same argument types

Reported by: strk Owned by: strk
Priority: blocker Milestone: PostGIS 3.4.0
Component: build Version: 3.2.x
Keywords: Cc:

Description

Passing a dump of postgis 3.0.0dev on PostgreSQL 13 to postgis_restore.pl fail with:

ERROR:  function "box2d_out" already exists with same argument types

The SKIP list contains box2d_out(box2d) but the manifest contains box2d_out(public.box2d) so the function is kept.

Change History (9)

comment:1 by robe, 3 years ago

Component: postgisbuild/upgrade/install
Owner: changed from pramsey to strk

comment:2 by strk, 3 years ago

Merge request https://gitlab.com/postgis/postgis/-/merge_requests/68 puts postgis_restore.pl under regression testing, which shows the failure

comment:3 by robe, 2 years ago

Milestone: PostGIS 3.3.0PostGIS 3.4.0

comment:4 by strk, 2 years ago

Pull request moved to OSGeo machines where CI still works: https://git.osgeo.org/gitea/postgis/postgis/pulls/112

comment:5 by strk, 2 years ago

Fails on self-dump too (current master dump, current master restore)

comment:6 by Sandro Santilli <strk@…>, 21 months ago

In ecea1b9/git:

run_test.pl: run before-upgrade hooks before dump, use postgis_restore.pl when needed

We can only use pg_restore when the dumped database is extension based.

See #5075 and #5338

comment:7 by strk, 21 months ago

Priority: mediumblocker

This is now confirmed by gitlab-ci, running dumpreload test using pg_restore: https://gitlab.com/strk/postgis/-/jobs/3785079800#L4124

comment:9 by Sandro Santilli <strk@…>, 21 months ago

Resolution: fixed
Status: newclosed

In 3e17616/git:

postgis_restore.pl: update skip list

Strip schema.
Closes #5075

Note: See TracTickets for help on using tickets.