From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [COMMITTERS] pgsql: Refactor NUM_cache_remove calls in error report path to a PG_TRY |
Date: | 2009-08-10 22:03:11 |
Message-ID: | 20090810220311.GH4796@alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Robert Haas escribió:
> But if there are patches against that code, then they've been broken
> now and they will break again when the pgindent run is done. If the
> indentation is fixed at commit-time (or before someone goes to the
> trouble of fixing them), then they get broken only once. I guess it's
> not the end of the world, but it sure seems like the less work
> pgindent does when it is run, the better.
I think that we should be looking at making pgindent runnable by lone
hackers at home in their patched trees. That way they fix their patches
by simply running it when it's run on the CVS tree, before doing an
"update". That should remove/reduce the merge problems.
... at least in CVS; not sure what would happen if this was done in a
GIT repository. (It would probably require a rebase, but then what do I
know about GIT?).
BTW I think it's better to redirect this kind of discussion to -hackers.
--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2009-08-10 22:13:50 | pgsql: Fix number of columns declared for pg_user_mappings description |
Previous Message | Robert Haas | 2009-08-10 21:39:30 | Re: pgsql: Refactor NUM_cache_remove calls in error report path to a PG_TRY |
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2009-08-10 22:15:51 | Re: hot standby - merged up to CVS HEAD |
Previous Message | Robert Haas | 2009-08-10 21:39:30 | Re: pgsql: Refactor NUM_cache_remove calls in error report path to a PG_TRY |