diff options
| author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2009-12-23 12:23:59 +0000 |
|---|---|---|
| committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2009-12-23 12:23:59 +0000 |
| commit | 4e766f2d176af92a8372b28a9ec2aa1eb8083063 (patch) | |
| tree | 9b61b7489abe228b19aa85c4ad1ce408afa527e5 /doc/src/sgml/ref/create_foreign_data_wrapper.sgml | |
| parent | b683908064e1fa4db26348ca43081dc6a99e3e4e (diff) | |
| download | postgresql-4e766f2d176af92a8372b28a9ec2aa1eb8083063.tar.gz postgresql-4e766f2d176af92a8372b28a9ec2aa1eb8083063.zip | |
Always pass catalog id to the options validator function specified in
CREATE FOREIGN DATA WRAPPER. Arguably it wasn't a bug because the
documentation said that it's passed the catalog ID or zero, but surely
we should provide it when it's known. And there isn't currently any
scenario where it's not known, and I can't imagine having one in the
future either, so better remove the "or zero" escape hatch and always
pass a valid catalog ID. Backpatch to 8.4.
Martin Pihlak
Diffstat (limited to 'doc/src/sgml/ref/create_foreign_data_wrapper.sgml')
| -rw-r--r-- | doc/src/sgml/ref/create_foreign_data_wrapper.sgml | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/doc/src/sgml/ref/create_foreign_data_wrapper.sgml b/doc/src/sgml/ref/create_foreign_data_wrapper.sgml index ee8b619a40e..b9ceb7ecc17 100644 --- a/doc/src/sgml/ref/create_foreign_data_wrapper.sgml +++ b/doc/src/sgml/ref/create_foreign_data_wrapper.sgml @@ -1,5 +1,5 @@ <!-- -$PostgreSQL: pgsql/doc/src/sgml/ref/create_foreign_data_wrapper.sgml,v 1.5 2009/06/19 15:28:25 petere Exp $ +$PostgreSQL: pgsql/doc/src/sgml/ref/create_foreign_data_wrapper.sgml,v 1.6 2009/12/23 12:23:58 heikki Exp $ PostgreSQL documentation --> @@ -74,10 +74,9 @@ CREATE FOREIGN DATA WRAPPER <replaceable class="parameter">name</replaceable> take two arguments: one of type <type>text[]</type>, which will contain the array of options as stored in the system catalogs, and one of type <type>oid</type>, which will be the OID of the - system catalog containing the options, or zero if the context is - not known. The return type is ignored; the function should - indicate invalid options using - the <function>ereport()</function> function. + system catalog containing the options. The return type is ignored; + the function should indicate invalid options using the + <function>ereport()</function> function. </para> </listitem> </varlistentry> |