Re: change name of redirect_stderr?

Bruce Momjian wrote:
> Andrew Dunstan wrote:
> > 
> > 
> > Alvaro Herrera wrote:
> > >>> That sounds like you want to log when the collector starts, which is not
> > >>> the case and is confusing -- what collector is it talking about?  This
> > >>> is about starting the log collector.
> > >>>       
> > >> Yea, good point.  I was just wondering because I don't see 'start' used
> > >> in anywhere at the beginning of a GUC variable.
> > >
> > > Good point too.  In other places we just name the feature that's to be
> > > started, for example we don't use "start_autovacuum".
> > 
> > How about just "log_collector" then?
> > 
> > Lets decide ASAP please - I want to get this committed.
> 
> Works for me, or enable_log_collector?  Based on Alvaro's comments,
> log_collector does sound like we are logging collector activity.

The problem here is that "log" seems to be a verb in "log_collector"
which is what makes it confusing.  So we need another verb to make it
clear that "log" is not one.  This is not a problem with "autovacuum"
because that one cannot be confused with a verb.

start_log_collector still gets my vote.

-- 
Alvaro Herrera                                http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.