public inbox for [email protected]  

help / color / mirror / Atom feed

From: Charles Clavadetscher <[email protected]>
To: 'Stephen Frost' <[email protected]>
Cc: [email protected]
Subject: Re: [DOCS] Missing COMMENT ON POLICY
Date: Sun, 13 Sep 2015 14:42:24 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<[email protected]>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgsql-hackers>

Hello Stephen

Yes, of course. I will take care of that and send a new patch.
Who will then git push it?

Thank you for your feedback.
Bye
Charles

> -----Original Message-----
> From: [email protected] [mailto:pgsql-hackers-
> [email protected]] On Behalf Of Stephen Frost
> Sent: Sonntag, 13. September 2015 14:31
> To: Charles Clavadetscher <[email protected]>
> Cc: [email protected]
> Subject: Re: [HACKERS] [DOCS] Missing COMMENT ON POLICY
> 
> Charles,
> 
> * Charles Clavadetscher ([email protected]) wrote:
> > Well I gave it a try. I am still not very well aware how the process for
> > suggesting or submitting corrections work, but this is as far as I got.
> > In the attachment is a patch for the missing entry for POLICY in the
> > documentation on COMMENT that I mentioned earlier on pgsql-docs.
> 
> Thanks!  This looks mostly correct but only goes half-way (not that
> there's a lot to this change ;).  The 'policy_name' parameter which is
> added should also be added to the 'Parameters' list below, and the
> description under 'table_name' updated to reflect that it's used for
> policies also.
> 
> There should probably be an example added also, since it looks like
> there are examples of pretty much all the other COMMENT ON
> types.
> 
> Thanks again!
> 
> Stephen



-- 
Sent via pgsql-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

view thread (11+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: [DOCS] Missing COMMENT ON POLICY
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox