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-23 16:19:59 |
Message-ID: | 32055c4d-fc0e-fb4d-6c31-5a4d084e4fda@oss.nttdata.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2021/04/23 19:56, Bharath Rupireddy wrote:
> On Fri, Apr 23, 2021 at 1:39 PM Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> wrote:
>>> + <para>
>>> + Note that information about <literal>ONLY</literal> options specified
>>> + in the original <command>TRUNCATE</command> command is not passed to
>>>
>>> I think it is not "information about", no? We just don't pass ONLY
>>> option instead we skip it. IMO, we can say "Note that the ONLY option
>>> specified with a foreign table in the original TRUNCATE command is
>>> skipped and not passed to ExecForeignTruncate."
>>
>> Probably I still fail to understand your point.
>> But if "information about" is confusing, I'm ok to
>> remove that. Fixed.
>
> A small typo in the docs patch: It is "are not passed to", instead of
> "is not passed to" since we used plural "options". "Note that the ONLY
> options specified in the original TRUNCATE command are not passed to"
>
> + Note that the <literal>ONLY</literal> options specified
> in the original <command>TRUNCATE</command> command is not passed to
Thanks for the review! I fixed this.
Regards,
--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION
Attachment | Content-Type | Size |
---|---|---|
truncate_foreign_table_docs_v4.patch | text/plain | 4.3 KB |
truncate_foreign_table_dont_pass_only_clause_v5.patch | text/plain | 17.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2021-04-23 17:04:52 | Re: A test for replay of regression tests |
Previous Message | Tom Lane | 2021-04-23 15:56:35 | Re: [bug?] Missed parallel safety checks, and wrong parallel safety |