Re: pgsql: Support comments on FOREIGN DATA WRAPPER and SERVER objects.

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

In response to

Responses

Browse pgsql-committers by date

  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

Browse pgsql-hackers by date

  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