From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Minor comment improvements in tablecmds.c |
Date: | 2014-02-25 18:44:27 |
Message-ID: | 20140225184427.GY4759@eldon.alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Etsuro Fujita wrote:
> This is a small patch to improve comments in tablecmds.c. Please find
> attached a patch.
I find both patched and unpatched to be pretty illegible. How about
something like
> /*
> - * Execute ALTER TABLE/INDEX/SEQUENCE/VIEW/FOREIGN TABLE RENAME
> + * Execute ALTER <relation type> RENAME
> + * This routine supports tables, indexes, sequences, views,
> + * and foreign tables
> */
and the two other ones with something like this:
> /*
> - * Grab an exclusive lock on the target table, index, sequence or view,
> - * which we will NOT release until end of transaction.
> + * Grab an exclusive lock on the target relation,
> + * which we will NOT release until
> + * end of transaction.
--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Josh Berkus | 2014-02-25 18:45:20 | Re: jsonb and nested hstore |
Previous Message | Robert Haas | 2014-02-25 18:41:31 | Re: Minor comment improvements in tablecmds.c |