Re: TRUNCATE on foreign table

From: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>
To: Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>
Cc: Kohei KaiGai <kaigai(at)heterodb(dot)com>, Kazutaka Onishi <onishi(at)heterodb(dot)com>, Zhihong Yu <zyu(at)yugabyte(dot)com>, Amit Langote <amitlangote09(at)gmail(dot)com>, Ibrar Ahmed <ibrar(dot)ahmad(at)gmail(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-04-15 14:49:43
Message-ID: a1975489-a32d-dc2b-c117-cd36495d82fa@oss.nttdata.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2021/04/14 12:54, Bharath Rupireddy wrote:
> IMHO, we can push all the TRUNCATE options (ONLY, RESTRICTED, CASCADE,
> RESTART/CONTINUE IDENTITY), because it doesn't have any major
> challenge(implementation wise) unlike pushing some clauses in
> SELECT/UPDATE/DELETE and we already do this on the master. It doesn't
> look good and may confuse users, if we push some options and restrict
> others. We should have an explicit note in the documentation saying we
> push all these options to the remote server. We can leave it to the
> user to write TRUNCATE for foreign tables with the appropriate
> options. If somebody complains about a problem that they will face
> with this behavior, we can revisit.

That's one of the options. But I'm afraid it's hard to drop (revisit)
the feature once it has been released. So if there is no explicit
use case for that, basically I'd like to drop that before release
like we agree to drop unused TRUNCATE_REL_CONTEXT_CASCADING.

Regards,

--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2021-04-15 15:01:51 Re: Commit ab596105b55 - BRIN minmax-multi indexes
Previous Message Tomas Vondra 2021-04-15 14:39:50 Re: Commit ab596105b55 - BRIN minmax-multi indexes