On Wed, May 25, 2022 at 9:44 PM Robert Haas <robertmh...@gmail.com> wrote: > On Sat, May 21, 2022 at 9:09 AM Justin Pryzby <pry...@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
PG10-missing-partition_bound_spec.patch
Description: Binary data
missing-partition_bound_spec.patch
Description: Binary data