On Tue, Jul 7, 2015 at 6:24 PM, Petr Jelinek <p...@2ndquadrant.com> wrote: > On 2015-07-04 13:45, Michael Paquier wrote: >> >> On Fri, Jul 3, 2015 at 11:59 PM, Petr Jelinek wrote: >>> >>> Well for indexes you don't really need to add the new AT command, as >>> IndexStmt has char *idxcomment which it will automatically uses as >>> comment >>> if not NULL. While I am not huge fan of the idxcomment it doesn't seem >>> to >>> be easy to remove it in the future and it's what transformTableLikeClause >>> uses so it might be good to be consistent with that. >> >> >> Oh, right, I completely missed your point and this field in IndexStmt. >> Yes it is better to be consistent in this case and to use it. It makes >> as well the code easier to follow. >> Btw, regarding the new AT routines, I am getting find of them, it >> makes easier to follow which command is added where in the command >> queues. >> >> Updated patch attached. >> > > Cool, I am happy with the patch now. Marking as ready for committer.
Thanks for the review. -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers