From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: ahsan hadi <ahsan.hadi@gmail.com>
Cc: pgsql-hackers@lists.postgresql.org
Subject: Re: more ALTER .. DEPENDS ON EXTENSION fixes
Date: Fri, 28 Feb 2020 10:44:51 -0300
Message-ID: <20200228134451.GA22878@alvherre.pgsql> (raw)
In-Reply-To: <158287616306.14910.17791108224723819457.pgcf@coridan.postgresql.org>
On 2020-Feb-28, ahsan hadi wrote:
> Tested the pg_dump patch for dumping "ALTER .. DEPENDS ON EXTENSION" in case of indexes, functions, triggers etc. The "ALTER .. DEPENDS ON EXTENSION" is included in the dump. However in some case not sure why "ALTER INDEX.....DEPENDS ON EXTENSION" is repeated several times in the dump?
Hi, thanks for testing.
Are the repeated commands for the same index, same extension? Did you
apply the same command multiple times before running pg_dump?
There was an off-list complaint that if you repeat the ALTER .. DEPENDS
for the same object on the same extension, then the same dependency is
registered multiple times. (You can search pg_depend for "deptype = 'x'"
to see that). I suppose that would lead to the line being output
multiple times by pg_dump, also. Is that what you did?
If so: Patch 0002 is supposed to fix that problem, by raising an error
if the dependency is already registered ... though it occurs to me now
that it would be more in line with custom to make the command a silent
no-op. In fact, doing that would cause old dumps (generated with
databases containing duplicated entries) to correctly restore a single
entry, without error. Therefore my inclination now is to change 0002
that way and push and backpatch it ahead of 0001.
I realize just now that I have failed to verify what happens with
partitioned indexes.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: alvherre@2ndquadrant.com, ahsan.hadi@gmail.com, pgsql-hackers@lists.postgresql.org
Subject: Re: more ALTER .. DEPENDS ON EXTENSION fixes
In-Reply-To: <20200228134451.GA22878@alvherre.pgsql>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox