From: Tom Lane <tgl@sss.pgh.pa.us>
To: Fujii Masao <masao.fujii@gmail.com>
cc: Andrei Klychkov <andrew.a.klychkov@gmail.com>,
        Jim Jones <jim.jones@uni-muenster.de>,
        pgsql-hackers@lists.postgresql.org
Subject: Re: [PATCH] Fix ALTER SYSTEM empty string bug for GUC_LIST_QUOTE
 parameters
In-reply-to: 
 <CAHGQGwEqxZ81JrJ+fO=8sYGFYTgrZ-M6XYErdLoiy8p82EXS-A@mail.gmail.com>
References: 
 <CA+mfrmwsBmYsJayWjc8bJmicxc3phZcHHY=yW5aYe=P-1d_4bg@mail.gmail.com>
 <38091678-2820-4bc8-8df7-b417562f7860@uni-muenster.de>
 <CA+mfrmxtk+f3dE8zJ2=5cWrbSw6YzOXNOmbi1iC0Hb0HCa-G7w@mail.gmail.com>
 <CAHGQGwG7ADhPorp0-29LGz0Vct=Dhhcerb8LdX2s+QC7TY=RRA@mail.gmail.com>
 <CA+mfrmwJgd3N1Lcqy64Fi6R+JHAT=bhf6rdh52hMRH78pBM5mQ@mail.gmail.com>
 <CAHGQGwEqxZ81JrJ+fO=8sYGFYTgrZ-M6XYErdLoiy8p82EXS-A@mail.gmail.com>
Comments: In-reply-to Fujii Masao <masao.fujii@gmail.com>
	message dated "Fri, 05 Sep 2025 01:42:06 +0900"
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-ID: <1046094.1757007639.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Thu, 04 Sep 2025 13:40:39 -0400
Message-ID: <1046095.1757007639@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/1046095.1757007639%40sss.pgh.pa.us>
Precedence: bulk

Fujii Masao <masao.fujii@gmail.com> writes:
>>> Even with this patch, an empty string set via SET is still quoted. For=
 example:
>>> =

>>> =3D# SET local_preload_libraries TO '';
>>> SET
>>> =3D# SHOW local_preload_libraries ;
>>> local_preload_libraries
>>> -------------------------
>>> ""
>>> (1 row)
>>> =

>>> Is this behavior acceptable? I was thinking that an empty string shoul=
d not
>>> be quoted, regardless of whether it's set by ALTER SYSTEM SET or SET.

> BTW, regarding the behavior change, I believe that users likely expect
> the parameter to be reset when specifying an empty string, rather than
> being set to "". So the proposed change seems reasonable. However,
> the current behavior has existed for a long time, and I haven=E2=80=99t =
seen
> any complaints about it.

I think this is largely based on confusion.  In the above example,
local_preload_libraries is being set to a list containing a single
entry that is an empty string, and the output of SHOW is a fully
accurate depiction of that state.  It is *not* being set to an
empty list --- we actually don't have any syntax that would permit
doing so in SET.  For comparison, there is a big difference between

	SET local_preload_libraries =3D a, b;
	SET local_preload_libraries =3D 'a, b';

In the latter case you get a single list entry containing the
string "a, b".  We do not try to parse that into multiple entries,
and by the same token parsing an empty string into an empty list
would be the Wrong Thing.

We might want to start resolving this by inventing a syntax for
setting a list GUC to an empty list.  I'm not very sure what that
should look like, except that it mustn't be SET ... TO ''.

I'm not certain whether config-file parsing or ALTER SYSTEM
would need any code changes once we resolve the ambiguity in SET.
The config-file syntax is different and doesn't have this problem
of not being able to represent an empty list.

(Also, "let's unify the list-GUC syntax between config file and SET"
seems like a non-starter.  It'd be better no doubt if they hadn't
diverged, but at this point we'd break far more than we fix if
we change either one.)

			regards, tom lane