Обсуждение: BUG #6152: possibly wrong display of pg_settings.enumvals for default_transaction_isolation

The following bug has been logged online:

Bug reference:      6152
Logged by:          Christoph Anton Mitterer
Email address:      calestyo@scientia.net
PostgreSQL version: 9.0
Operating system:   Debian sid
Description:        possibly wrong display of pg_settings.enumvals for
default_transaction_isolation
Details:

Hi

$ psql -c 'SELECT name,vartype,enumvals FROM pg_settings;' | grep
default_transaction_isolation
could not change directory to "/root"
 default_transaction_isolation   | enum    | {serializable,"repeatable
read","read committed","read uncommitted"}

Using e.g.:
default_transaction_isolation = "read committed"
does not work however. It only works in single quotes.

If I don't get something wrong, there should be either a bug in the output
of pg_settings.enumvals or the parsing of the config file.


Cheers,
Chris.

"Christoph Anton Mitterer" <calestyo@scientia.net> writes:
> $ psql -c 'SELECT name,vartype,enumvals FROM pg_settings;' | grep
> default_transaction_isolation
>  default_transaction_isolation   | enum    | {serializable,"repeatable
> read","read committed","read uncommitted"}

That output looks correct to me.  The column is an array column, and
double quotes are needed to protect the whitespace embedded in the last
three array elements.  (Strictly speaking they could perhaps be omitted,
but the array output function is a bit conservative here --- any
whitespace causes it to include quotes.)

> Using e.g.:
> default_transaction_isolation = "read committed"
> does not work however. It only works in single quotes.

Yeah, the configuration file parser only allows single quotes.
I suppose we could let it accept double quotes too, but I'd be worried
about foreclosing future expansion.  The two kinds of quotes mean
entirely different things in SQL commands.

            regards, tom lane

Hi.

ok,.. I got that wrong and thought the enumvals was intended to be the
exact list how to set the values in the config.


Cheers,
Chris.