Re: pg_restore with --disable-triggers discards ENABLE ALWAYS - Mailing list pgsql-bugs

From Laurenz Albe
Subject Re: pg_restore with --disable-triggers discards ENABLE ALWAYS
Date
Msg-id 78a13f951a03bf325027f8dcfb32b1b5089cdb14.camel@cybertec.at
Whole thread Raw
In response to pg_restore with --disable-triggers discards ENABLE ALWAYS  (Duncan Sands <duncan.sands@deepbluecap.com>)
Responses Re: pg_restore with --disable-triggers discards ENABLE ALWAYS
Re: pg_restore with --disable-triggers discards ENABLE ALWAYS
List pgsql-bugs
On Thu, 2024-09-12 at 10:27 +0200, Duncan Sands wrote:
> CREATE TABLE test_table(x int);
> CREATE FUNCTION test_function() RETURNS trigger AS $$ BEGIN RETURN NULL; END; $$
> LANGUAGE plpgsql;
> CREATE TRIGGER test_trigger AFTER UPDATE ON test_table FOR EACH ROW EXECUTE
> FUNCTION test_function();
> ALTER TABLE test_table ENABLE ALWAYS TRIGGER test_trigger;
>
> Checking the table:
>
> duncan=> \d test_table
>               Table "public.test_table"
>   Column |  Type   | Collation | Nullable | Default
> --------+---------+-----------+----------+---------
>   x      | integer |           |          |
> Triggers firing always:
>      test_trigger AFTER UPDATE ON test_table FOR EACH ROW EXECUTE FUNCTION
> test_function()
>
> ^ Observe "Triggers firing always".
>
>
> Now for the dump + restore:
>
> pg_dump -f dump.custom -Fc --table test_table
> pg_restore --data-only --disable-triggers --dbname duncan dump.custom
>
> Checking the table:
>
> duncan=> \d test_table
>               Table "public.test_table"
>   Column |  Type   | Collation | Nullable | Default
> --------+---------+-----------+----------+---------
>   x      | integer |           |          |
> Triggers:
>      test_trigger AFTER UPDATE ON test_table FOR EACH ROW EXECUTE FUNCTION
> test_function()
>
> ^ Observe that "Triggers firing always" has disappeared.

This looks like a user error to me.
If you restore with --data-only, the table and constraint definitions
are not restored at all.  So the table "test_table" in database "duncan"
must already have existed before the restore, and the trigger was already
defined like that.

Yours,
Laurenz Albe



pgsql-bugs by date:

Previous
From: Tom Lane
Date:
Subject: Re: BUG #18613: Incorrect output for foreign tables with NOT NULL constraints
Next
From: Tom Lane
Date:
Subject: Re: pg_restore with --disable-triggers discards ENABLE ALWAYS