On 7 June 2012 22:54, Tom Lane <t...@sss.pgh.pa.us> wrote: > I thought it would be a lot safer and probably a little bit quicker > if we just split DropRelFileNodeBuffers into two routines, one for > the specific-fork case and one for the all-forks case; and then the > same for its main caller smgrdounlink. So I modified the patch along > those lines and committed it. > > As committed, the smgrdounlinkfork case is actually dead code; it's > never called from anywhere. I left it in place just in case we want > it someday.
That's fine. The first version of the patch did it exactly that way. I tried to double guess objections and so recoded it the way submitted. -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers