On Thu, Jan 19, 2017 at 7:27 AM, Alvaro Herrera
<alvherre@2ndquadrant.com> wrote:
> I thought this part was odd -- I mean, why is SysLogger_Start() being
> called if the collector is not enabled? Turns out we do it and return
> early if not enabled. But not in all cases -- there is one callsite in
> postmaster.c that avoids the call if the collector is disabled. That
> needs to be changed if we want this to work reliably.
Indeed. And actually it is fine to remove the call to FreeFile() in
the error code path of pg_current_logfile().
> I don't think the "abstract names" stuff is an improvement (just look at
> the quoting mess in ConfigureNamesString). I think we should do without
> that; at least as part of this patch. If you think there's code that
> can get better because of the idea, let's see it.
Agreed.
--
Michael