Re: [HACKERS] change name of redirect_stderr?

2007-08-18 Thread Andrew Dunstan
Tom Lane wrote: Brendan Jurd [EMAIL PROTECTED] writes: On 8/15/07, Tom Lane [EMAIL PROTECTED] wrote: For example, log_line_prefix is misnamed under this rule, and ought to be logging_line_prefix. Similarly, redirect_stderr would become logging_something --- I'd prefer

Re: [HACKERS] change name of redirect_stderr?

2007-08-18 Thread Michael Glaesemann
On Aug 18, 2007, at 20:44 , Andrew Dunstan wrote: Logging_collector won the day. I have just committed CSVlogs with that change. Congrats! A couple last-minute correx: http://developer.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ func.sgml?r1=1.385r2=1.386 s/log collector if

Re: [HACKERS] change name of redirect_stderr?

2007-08-18 Thread Andrew Dunstan
Michael Glaesemann wrote: On Aug 18, 2007, at 20:44 , Andrew Dunstan wrote: Logging_collector won the day. I have just committed CSVlogs with that change. Congrats! A couple last-minute correx: thanks. fixed. cheers andrew ---(end of

Re: [HACKERS] change name of redirect_stderr?

2007-08-15 Thread Josh Berkus
Heikki, Should we change the ordering of pg_settings? Well, an unfixed issue in pg_settings is that the category/subcategory relationship got represented by a non-atomic field which makes sorting on that difficult. -- --Josh Josh Berkus PostgreSQL @ Sun San Francisco

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Bruce Momjian
Andrew Dunstan wrote: Tom Lane wrote: Andrew Dunstan [EMAIL PROTECTED] writes: Before I wrap up the CSVlog stuff, we need to decide whether or not to change the name of the redirect_stderr setting, and if so to what. The reason is that with CSVlogs it will no longer apply just

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Alvaro Herrera
Bruce Momjian wrote: Andrew Dunstan wrote: I suggest redirect_logs, although it's arguably too general as it doesn't apply to syslog/eventlog. Perhaps it should be named analogously to stats_start_collector, ie think of the syslogger process as a log collector. I don't much

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Bruce Momjian
Alvaro Herrera wrote: Bruce Momjian wrote: Andrew Dunstan wrote: I suggest redirect_logs, although it's arguably too general as it doesn't apply to syslog/eventlog. Perhaps it should be named analogously to stats_start_collector, ie think of the syslogger process as a

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Alvaro Herrera
Bruce Momjian wrote: Alvaro Herrera wrote: Bruce Momjian wrote: Andrew Dunstan wrote: I suggest redirect_logs, although it's arguably too general as it doesn't apply to syslog/eventlog. Perhaps it should be named analogously to stats_start_collector, ie think

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Andrew Dunstan
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

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Bruce Momjian
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

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Kevin Grittner
On Tue, Aug 14, 2007 at 11:42 AM, in message [EMAIL PROTECTED], Andrew Dunstan [EMAIL PROTECTED] wrote: Alvaro Herrera wrote: 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? +1 Unambiguous and

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Alvaro Herrera
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

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Josh Berkus
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.

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Tom Lane
Josh Berkus [EMAIL PROTECTED] writes: 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.

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Heikki Linnakangas
Josh Berkus wrote: 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

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Michael Glaesemann
On Aug 14, 2007, at 12:40 , Tom Lane wrote: Josh Berkus [EMAIL PROTECTED] writes: 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

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Tom Lane
Michael Glaesemann [EMAIL PROTECTED] writes: AIUI, if the-GUC-yet-to-be-named is not enabled, no logging is done at all: messages are just sent to stderr. Why something simple like enable_logging or start_logger? Um, that's still logging by my definition. I could live with start_logger,

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Andrew Dunstan
Heikki Linnakangas wrote: Josh Berkus wrote: 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

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Brendan Jurd
On 8/15/07, Tom Lane [EMAIL PROTECTED] wrote: For example, log_line_prefix is misnamed under this rule, and ought to be logging_line_prefix. Similarly, redirect_stderr would become logging_something --- I'd prefer logging_start_collector but could live with logging_collector (or maybe

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Alvaro Herrera
With so many people trying to paint this particular bikeshed, my suggestion to Andrew is to commit the patch as is and leave the rename for a later patch. -- Alvaro Herrera http://www.amazon.com/gp/registry/DXLWNGRJD34J Jude: I wish humans laid eggs Ringlord: Why would you want

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Tom Lane
Brendan Jurd [EMAIL PROTECTED] writes: On 8/15/07, Tom Lane [EMAIL PROTECTED] wrote: For example, log_line_prefix is misnamed under this rule, and ought to be logging_line_prefix. Similarly, redirect_stderr would become logging_something --- I'd prefer logging_start_collector but could live

Re: [HACKERS] change name of redirect_stderr?

2007-08-14 Thread Brendan Jurd
On 8/15/07, Tom Lane [EMAIL PROTECTED] wrote: Brendan Jurd [EMAIL PROTECTED] writes: The consistent prefix idea sounds good; does logging_enable jive with your proposal? I dislike it. I claim that logging to plain stderr (without the syslogger process) is still logging. Logging to syslog

Re: [HACKERS] change name of redirect_stderr?

2007-08-10 Thread Andrew Dunstan
Tom Lane wrote: Andrew Dunstan [EMAIL PROTECTED] writes: Before I wrap up the CSVlog stuff, we need to decide whether or not to change the name of the redirect_stderr setting, and if so to what. The reason is that with CSVlogs it will no longer apply just to stderr (we will require it to

Re: [HACKERS] change name of redirect_stderr?

2007-08-09 Thread Tom Lane
Andrew Dunstan [EMAIL PROTECTED] writes: Before I wrap up the CSVlog stuff, we need to decide whether or not to change the name of the redirect_stderr setting, and if so to what. The reason is that with CSVlogs it will no longer apply just to stderr (we will require it to be on for CSVlogs,