[9.1 beta 1] log_timezone = "unknown" does not work any more

Started by Martin Pittalmost 15 years ago3 messagesbugs
Jump to latest
#1Martin Pitt
mpitt@debian.org

Hello all,

I (or rather the Debian postgresql-common) test suite noticed that
postgresql.conf's "log_timezone = unknown" does not work any more:

server starting
FATAL: invalid value for parameter "log_timezone": "unknown"

I think this should be considered a regression, as it's still in
postgresql.conf.sample, and in the documentation [1]http://developer.postgresql.org/pgdocs/postgres/runtime-config-logging.html#GUC-LOG-TIMEZONE -- Martin Pitt | http://www.piware.de Ubuntu Developer (www.ubuntu.com) | Debian Developer (www.debian.org).

Thanks,

Martin

[1]: http://developer.postgresql.org/pgdocs/postgres/runtime-config-logging.html#GUC-LOG-TIMEZONE -- Martin Pitt | http://www.piware.de Ubuntu Developer (www.ubuntu.com) | Debian Developer (www.debian.org)
--
Martin Pitt | http://www.piware.de
Ubuntu Developer (www.ubuntu.com) | Debian Developer (www.debian.org)

#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Martin Pitt (#1)
Re: [9.1 beta 1] log_timezone = "unknown" does not work any more

Martin Pitt <mpitt@debian.org> writes:

Hello all,
I (or rather the Debian postgresql-common) test suite noticed that
postgresql.conf's "log_timezone = unknown" does not work any more:

That was an intentional change, actually. Do you have a real use case
for setting it that way?

I think this should be considered a regression, as it's still in
postgresql.conf.sample, and in the documentation [1].

Hm. I forgot to modify postgresql.conf.sample, and I'm not entirely
sure what to change it to. Maybe just do

#log_timezone = '(default is system's timezone)'

Anyone have a better idea?

regards, tom lane

#3Martin Pitt
mpitt@debian.org
In reply to: Tom Lane (#2)
Re: [9.1 beta 1] log_timezone = "unknown" does not work any more

Hello,

Tom Lane [2011-05-10 10:35 -0400]:

That was an intentional change, actually. Do you have a real use case
for setting it that way?

I don't, it's quite an useless setting. I just stumbled over it as I
use the standard upstream postgresql.conf.sample files (uncommented,
of course) in the self tests for configuration rewriting on major
version upgrade. I'm happy to change them to use a real value.

I think this should be considered a regression, as it's still in
postgresql.conf.sample, and in the documentation [1].

Hm. I forgot to modify postgresql.conf.sample, and I'm not entirely
sure what to change it to. Maybe just do

#log_timezone = '(default is system's timezone)'

FWIW, looks tood to me. This makes it a lot more obvious that this
isn't an accepted literal value.

Thanks!

Martin
--
Martin Pitt | http://www.piware.de
Ubuntu Developer (www.ubuntu.com) | Debian Developer (www.debian.org)