Re: doc: CREATE FOREIGN TABLE .. PARTITION OF .. DEFAULT - Mailing list pgsql-hackers

From Amit Langote
Subject Re: doc: CREATE FOREIGN TABLE .. PARTITION OF .. DEFAULT
Date
Msg-id CA+HiwqHaxj+xJkRjg8KGJ+bdMkat1gVSAye02dRvOaWfeBYn8w@mail.gmail.com
Whole thread Raw
In response to Re: doc: CREATE FOREIGN TABLE .. PARTITION OF .. DEFAULT  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: doc: CREATE FOREIGN TABLE .. PARTITION OF .. DEFAULT
List pgsql-hackers
On Wed, May 25, 2022 at 9:44 PM Robert Haas <robertmhaas@gmail.com> wrote:
> On Sat, May 21, 2022 at 9:09 AM Justin Pryzby <pryzby@telsasoft.com> wrote:
> > It looks like the docs weren't updated in 6f6b99d13 for v11.
>
> In my defense, that commit definitely contained documentation changes.
> It updated alter_table.sgml and create_table.sgml. I guess we missed
> create_foreign_table.sgml, though.
>
> > The docs also seem to omit "FOR VALUES" literal.

That would be my mistake.

> > And don't define partition_bound_spec (which I didn't fix here).
> >
> > diff --git a/doc/src/sgml/ref/create_foreign_table.sgml b/doc/src/sgml/ref/create_foreign_table.sgml
> > index b374d8645db..1f1c4a52a2a 100644
> > --- a/doc/src/sgml/ref/create_foreign_table.sgml
> > +++ b/doc/src/sgml/ref/create_foreign_table.sgml
> > @@ -35,7 +35,8 @@ CREATE FOREIGN TABLE [ IF NOT EXISTS ] <replaceable class="parameter">table_name
> >    { <replaceable class="parameter">column_name</replaceable> [ WITH OPTIONS ] [ <replaceable
class="parameter">column_constraint</replaceable>[ ... ] ]
 
> >      | <replaceable>table_constraint</replaceable> }
> >      [, ... ]
> > -) ] <replaceable class="parameter">partition_bound_spec</replaceable>
> > +) ]
> > +  { FOR VALUES <replaceable class="parameter">partition_bound_spec</replaceable> | DEFAULT }
> >    SERVER <replaceable class="parameter">server_name</replaceable>
> >  [ OPTIONS ( <replaceable class="parameter">option</replaceable> '<replaceable
class="parameter">value</replaceable>'[, ... ] ) ]
 
>
> OK, makes sense. I guess we need to copy over the definition of
> partition_bound_spec from create_table.sgml here as well.

Yes.  a2a2205761 did that for alter_table.sgml and we evidently missed
including create_foreign_table.sgml in that discussion.

Attached 2 patches -- one for PG 11 onwards and another for PG 10.

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

Attachment

pgsql-hackers by date:

Previous
From: Ajin Cherian
Date:
Subject: Re: Support logical replication of DDLs
Next
From: Gurjeet Singh
Date:
Subject: Re: Patch: Don't set LoadedSSL unless secure_initialize succeeds