From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pg_upgrade exit_nicely() |
Date: | 2011-03-31 20:54:15 |
Message-ID: | 201103312054.p2VKsFR28954@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Feel free to apply this to HEAD.
---------------------------------------------------------------------------
Peter Eisentraut wrote:
> While reading around in pg_upgrade code I came across the slightly
> bizarre function
>
> void exit_nicely(bool need_cleanup)
>
> The parameter doesn't actually determine whether any cleanup is done.
> The "cleanup" is done anyway and the parameter only controls the exit
> code in a backwards way.
>
> Also most of the cleanup appears to be useless, because you don't need
> to close files or free memory before the program exits.
>
> I figured this could be written more cleanly with an exit hook, so here
> is a patch. I don't care much whether this patch is for now or later,
> just wanted to throw it out there.
>
[ Attachment, skipping... ]
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2011-03-31 20:58:30 | Re: cast from integer to money |
Previous Message | AAMIR KHAN | 2011-03-31 20:38:22 | GSoC 2011 Eager MV implementation proposal |