From: | Thom Brown <thom(at)linux(dot)com> |
---|---|
To: | Robert Haas <rhaas(at)postgresql(dot)org> |
Cc: | pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Support comments on FOREIGN DATA WRAPPER and SERVER objects. |
Date: | 2011-04-01 15:57:11 |
Message-ID: | AANLkTi==NemsL7Vo=YfsPc8DdZActYF7ZwCFVO-0Nu21@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On 1 April 2011 16:28, Robert Haas <rhaas(at)postgresql(dot)org> wrote:
> Support comments on FOREIGN DATA WRAPPER and SERVER objects.
>
> This mostly involves making it work with the objectaddress.c framework,
> which does most of the heavy lifting. In that vein, change
> GetForeignDataWrapperOidByName to get_foreign_data_wrapper_oid and
> GetForeignServerOidByName to get_foreign_server_oid, to match the
> pattern we use for other object types.
>
> Robert Haas and Shigeru Hanada
>
> Branch
> ------
> master
>
> Details
> -------
> http://git.postgresql.org/pg/commitdiff/50533a6dc515cc3182f52838275c9d2a1f587604
>
> Modified Files
> --------------
> doc/src/sgml/ref/comment.sgml | 2 +
> src/backend/catalog/aclchk.c | 31 ++++++++++-
> src/backend/catalog/objectaddress.c | 33 +++++++++++-
> src/backend/commands/foreigncmds.c | 4 +-
> src/backend/foreign/foreign.c | 82 ++++++++++++++--------------
> src/backend/parser/gram.y | 13 +++--
> src/backend/utils/adt/acl.c | 4 +-
> src/include/foreign/foreign.h | 5 +-
> src/include/utils/acl.h | 1 +
> src/test/regress/expected/foreign_data.out | 2 +
> src/test/regress/sql/foreign_data.sql | 2 +
> 11 files changed, 124 insertions(+), 55 deletions(-)
Should we also have support for comments on user mappings?
--
Thom Brown
Twitter: @darkixion
IRC (freenode): dark_ixion
Registered Linux user: #516935
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2011-04-01 17:16:58 | pgsql: Tab completion for COMMENT ON FOREIGN DATA WRAPPER / SERVER. |
Previous Message | Robert Haas | 2011-04-01 15:41:21 | pgsql: pg_dump support for comments on FOREIGN DATA WRAPPER and SERVER |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2011-04-01 16:14:18 | Re: Bug in autovacuum.c? |
Previous Message | David E. Wheeler | 2011-04-01 15:45:02 | Extensions Dependency Checking |