Re: TRUNCATE on foreign table

From: Kazutaka Onishi <onishi(at)heterodb(dot)com>
To: Amit Langote <amitlangote09(at)gmail(dot)com>
Cc: Ibrar Ahmed <ibrar(dot)ahmad(at)gmail(dot)com>, Kohei KaiGai <kaigai(at)heterodb(dot)com>, Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Zhihong Yu <zyu(at)yugabyte(dot)com>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>
Subject: Re: TRUNCATE on foreign table
Date: 2021-03-13 03:35:46
Message-ID: CAJuF6cN4oU==vATJ7AgSqxCzV6X7j95O97LtKJcwddVcAcEx1w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

To Ibrar,
Thank you for updating the patch!

To Amit,
Thank you for checking the patch, and I have confirmed the failure.
Now I'm trying to fix it.

2021年3月9日(火) 11:54 Amit Langote <amitlangote09(at)gmail(dot)com>:

> On Tue, Mar 9, 2021 at 2:24 AM Ibrar Ahmed <ibrar(dot)ahmad(at)gmail(dot)com> wrote:
> > The patch (pgsql14-truncate-on-foreign-table.v2.patch) does not apply
> successfully.
> >
> > http://cfbot.cputube.org/patch_32_2972.log
> >
> > patching file contrib/postgres_fdw/expected/postgres_fdw.out
> > Hunk #2 FAILED at 9179.
> > 1 out of 2 hunks FAILED -- saving rejects to file
> contrib/postgres_fdw/expected/postgres_fdw.out.rej
> >
> > As this is a minor change therefore I have updated the patch. Please
> take a look.
>
> Thanks for updating the patch. I was able to apply it successfully
> though I notice it doesn't pass make check-world.
>
> Specifically, it fails the src/test/subscription/013_partition.pl
> test. The problem seems to be that worker.c: apply_handle_truncate()
> hasn't been updated to add entries to relids_extra for partitions
> expanded from a partitioned table, like ExecuteTruncate() does. That
> leads to relids and relids_extra having different lengths, which trips
> the Assert in ExecuteTruncateGuts().
>
> --
> Amit Langote
> EDB: http://www.enterprisedb.com
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Thomas Munro 2021-03-13 03:37:11 Re: pgbench: option delaying queries till connections establishment?
Previous Message Tom Lane 2021-03-13 03:08:55 Re: pgbench: option delaying queries till connections establishment?