Re: [PATCH] Introduce unified support for composite GUC options

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
Cc: "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, Чумак Антон <a(dot)chumak(at)postgrespro(dot)ru>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCH] Introduce unified support for composite GUC options
Date: 2025-09-23 03:50:40
Message-ID: 3014217.1758599440@sss.pgh.pa.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> writes:
> Using GUC as session variables is a workaround because there is nothing
> better. But it is not good solution

Agreed, but we don't yet have a better one ...

> The basic question is if variables should be typed or typeless - like
> plpgsql or psql variables.

I think it is absolutely critical that GUCs *not* depend on the
SQL type system in any way. That would be a fundamental layering
violation, because we need to be able to read postgresql.conf
before we can read catalogs --- not to mention that relevant type
definitions might be different in different databases.

I'm not sure that this point means much to the feature proposed in
this thread, since IIUC it's proposing "use JSON no matter what".
But it is a big problem for trying to use GUCs as session variables
with non-built-in types.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Stehule 2025-09-23 04:11:21 Re: [PATCH] Introduce unified support for composite GUC options
Previous Message Pavel Stehule 2025-09-23 03:50:16 Re: [PATCH] Introduce unified support for composite GUC options