Re: postgres_fdw: IMPORT FOREIGN SCHEMA ... LIMIT TO (partition)

From: Amit Langote <amitlangote09(at)gmail(dot)com>
To: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>
Cc: Bernd Helmle <mailings(at)oopsware(dot)de>, Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Stephen Frost <sfrost(at)snowman(dot)net>, Michael Paquier <michael(at)paquier(dot)xyz>
Subject: Re: postgres_fdw: IMPORT FOREIGN SCHEMA ... LIMIT TO (partition)
Date: 2021-04-06 07:05:47
Message-ID: CA+HiwqHjA390+6WGD29cLS4Xmvj_VBSWnBsu_L0gfN-QyzWitw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Apr 6, 2021 at 8:34 AM Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> wrote:
> For now I have no objection to this feature.
>
> -IMPORT FOREIGN SCHEMA import_source EXCEPT (t1, "x 4", nonesuch)
> +IMPORT FOREIGN SCHEMA import_source EXCEPT (t1, "x 4", nonesuch, t4_part)
>
> Isn't it better to create also another partition like "t4_part2"?
> If we do this, for example, the above test can confirm that both
> partitions in EXCEPT and not in are excluded.
>
> + All tables or foreign tables which are partitions of some other table
> + are automatically excluded from <xref linkend="sql-importforeignschema"/>
> + unless they are explicitly included in the <literal>LIMIT TO</literal>
>
> IMO it's better to document that partitions are imported when they are
> included in LIMIT TO, instead. What about the following?
>
> Tables or foreign tables which are partitions of some other table are
> imported only when they are explicitly specified in
> <literal>LIMIT TO</literal> clause. Otherwise they are automatically
> excluded from <xref linkend="sql-importforeignschema"/>.
>
> + clause. Since all data can be accessed through the partitioned table
> + which is the root of the partitioning hierarchy, this approach should
> + allow access to all the data without creating extra objects.
>
> Now "this approach" in the above is not clear? What about replacing it with
> something like "importing only partitioned tables"?

+1, that wording is better.

--
Amit Langote
EDB: http://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Langote 2021-04-06 07:46:26 Re: A reloption for partitioned tables - parallel_workers
Previous Message Amit Langote 2021-04-06 07:00:38 Re: postgres_fdw: IMPORT FOREIGN SCHEMA ... LIMIT TO (partition)