Re: [PATCH] Logical decoding of TRUNCATE

2018-01-25 Thread Marco Nenciarini
Il 25/01/18 13:18, Petr Jelinek ha scritto: > On 25/01/18 08:26, Thomas Munro wrote: >> On Wed, Jan 24, 2018 at 6:47 AM, Marco Nenciarini >> <marco.nenciar...@2ndquadrant.it> wrote: >>> Version 16 attached. >> >> Hi Marco, >> >> FYI thi

Re: [PATCH] Logical decoding of TRUNCATE

2018-01-23 Thread Marco Nenciarini
Il 23/01/18 18:13, Petr Jelinek ha scritto: > Hi, > > On 23/01/18 15:38, Marco Nenciarini wrote: >> Il 22/01/18 23:18, Petr Jelinek ha scritto: >>> On 22/01/18 19:45, Petr Jelinek wrote: >>> >>> Actually on second look, I don't like the new boolean para

Re: [PATCH] Logical decoding of TRUNCATE

2018-01-23 Thread Marco Nenciarini
gt; > It may mean more list(s) but the current interface is still not clean. > Now ExecuteTruncateGuts unconditionally closes the relations that it opens. The caller has now always the responsibility to close the relations passed with the explicit_rels list. Version 15 attached. Regards, Mar

Re: [PATCH] session_replication_role = replica with TRUNCATE

2018-01-23 Thread Marco Nenciarini
Il 22/01/18 19:41, Petr Jelinek ha scritto: > On 19/01/18 12:41, Marco Nenciarini wrote: >> Hi Peter, >> >> Il 18/01/18 17:30, Peter Eisentraut ha scritto: >>> On 1/17/18 11:33, Petr Jelinek wrote: >>>>> P.S: I'm struggling

pg_upgrade tests failing on current master

2018-01-23 Thread Marco Nenciarini
t; ("ename", "who", "action", "newsal", "oldsal") VALUES ("old"."ename", CURRENT_USER, 'fired'::"bpchar", '$0.00'::"money", "old"."salary"); I'm on OSX with locale "it_IT.UTF-8&q

Re: [PATCH] session_replication_role = replica with TRUNCATE

2018-01-19 Thread Marco Nenciarini
and not replicated changes. > > I have committed some documentation updates and tests to cover this a > bit better. > Thanks, the documentation is a lot clearer now. This superseded the documentation change that was in the patch, so I've removed it from the v3 version. Regards, Marco -

Re: [PATCH] Logical decoding of TRUNCATE

2018-01-19 Thread Marco Nenciarini
ts should track >> which relations it opened and only close those and the rest should be >> closed by caller? That should also remove the other ugly part which is >> that the ExecuteTruncateGuts modifies the input list. What if caller >> wanted to use those relations it sent as parameter later? > &

Re: [PATCH] Logical decoding of TRUNCATE

2018-01-06 Thread Marco Nenciarini
Attached here there is the complete list of patches required to pass all the tests. The 0001 patch is discussed in a separate thread, but I've posted it also here to ease the review of the 0002. Regards, Marco -- Marco Nenciarini - 2ndQuadrant Italy PostgreSQL Training, Services and Support

Re: [PATCH] Logical decoding of TRUNCATE

2018-01-06 Thread Marco Nenciarini
Patch rebased on the current master. Regards, Marco -- Marco Nenciarini - 2ndQuadrant Italy PostgreSQL Training, Services and Support marco.nenciar...@2ndquadrant.it | www.2ndQuadrant.it diff --git a/contrib/test_decoding/expected/ddl.out b/contrib/test_decoding/expected/ddl.out index

Re: [PATCH] Logical decoding of TRUNCATE

2018-01-05 Thread Marco Nenciarini
Hi, I've found some SGML errors in the version v10 of the patch. I've fixed it in version v11 that is attached. Regards, Marco -- Marco Nenciarini - 2ndQuadrant Italy PostgreSQL Training, Services and Support marco.nenciar...@2ndquadrant.it | www.2ndQuadrant.it diff --git a/contrib

Re: [PATCH] session_replication_role = replica with TRUNCATE

2018-01-02 Thread Marco Nenciarini
Hi, Il 30/12/17 08:42, Craig Ringer ha scritto: > On 30 December 2017 at 03:32, Petr Jelinek <petr.jeli...@2ndquadrant.com > <mailto:petr.jeli...@2ndquadrant.com>> wrote: > > On 29/12/17 16:53, Marco Nenciarini wrote: > > Il 29/12/17 15:14, Petr Jelinek ha

Re: [PATCH] session_replication_role = replica with TRUNCATE

2017-12-29 Thread Marco Nenciarini
foreign keys or referential integrity, but only about triggers and rules. I don't think that it's worth to add a special case for truncate, unless we want to expand/rewrite the documentation to specify all the effects in the details. Regards, Marco -- Marco Nenciarini - 2ndQuadrant Italy PostgreSQL Traini

[PATCH] Logical decoding of TRUNCATE

2017-12-29 Thread Marco Nenciarini
when session_replication_role = replica"[1] patch is required from this patch to work correctly with tables referenced by foreign keys. [1] https://commitfest.postgresql.org/16/1447/ Regards, Marco -- Marco Nenciarini - 2ndQuadrant Italy PostgreSQL Training, Services and Support mar

[PATCH] session_replication_role = replica with TRUNCATE

2017-12-29 Thread Marco Nenciarini
. I'm attaching a simple patch to make TRUNCATE match behavior of DELETE for session_replication_role = replica. Regards, Marco -- Marco Nenciarini - 2ndQuadrant Italy PostgreSQL Training, Services and Support marco.nenciar...@2ndquadrant.it | www.2ndQuadrant.it diff --git a/src/backend/commands

Re: pspg - psql pager

2017-11-17 Thread Marco Nenciarini
he NEW queue for manual review from Debian's FTP Master. https://ftp-master.debian.org/new/pspg_0.5-1.html Regards, Marco -- Marco Nenciarini - 2ndQuadrant Italy PostgreSQL Training, Services and Support marco.nenciar...@2ndquadrant.it | www.2ndQuadrant.it signature.asc Description: OpenPGP digital signature