From: | Álvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
---|---|
To: | Zhang Mingli <zmlpostgres(at)gmail(dot)com> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Proposal to CREATE FOREIGN TABLE LIKE |
Date: | 2025-02-06 10:31:43 |
Message-ID: | 202502061031.wlj7ysbk7rfi@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2025-Feb-06, Álvaro Herrera wrote:
> On 2025-Feb-06, Zhang Mingli wrote:
>
> > Added support for CREATE FOREIGN TABLE LIKE to enable the creation of
> > foreign tables based on the column definitions, constraints of
> > existing source tables.
> > This feature mirrors the behavior of CREATE TABLE LIKE, but ignores
> > inapplicable options such as INCLUDING INDEXES and INCLUDING
> > COMPRESSION for foreign tables.
>
> I think it'd be better to throw errors if they are given -- but
> INCLUDING ALL should be made to work in a different way than today so
> that it doesn't raise errors uselessly. Right now it works by setting
> all the bits in the value, um.
Ah, but our fine manual already says
The LIKE clause can also be used to copy column definitions from views,
foreign tables, or composite types. Inapplicable options (e.g.,
INCLUDING INDEXES from a view) are ignored.
so what you implemented seems to be okay from that POV.
--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/
"Investigación es lo que hago cuando no sé lo que estoy haciendo"
(Wernher von Braun)
From | Date | Subject | |
---|---|---|---|
Next Message | Nisha Moond | 2025-02-06 10:38:17 | Re: Introduce XID age and inactive timeout based replication slot invalidation |
Previous Message | Álvaro Herrera | 2025-02-06 10:29:18 | Re: Proposal to CREATE FOREIGN TABLE LIKE |