Re: Improve documentation for current_setting function

From: Eugen Konkov <kes-kes(at)yandex(dot)ru>
To: Bruce Momjian <bruce(at)momjian(dot)us>, pgsql-docs(at)lists(dot)postgresql(dot)org
Subject: Re: Improve documentation for current_setting function
Date: 2019-02-22 09:43:54
Message-ID: 925065522.20190222114354@yandex.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs

Hello Bruce,

> How do you like the attached patch, which clarifies this?

>>in which case a missing parameter returns
>>! NULL.

not always. It can be NULL or empty string. It depends if the current
session is new or after finished transaction which makes 'local' changes
to setting

Take into account these steps: https://www.postgresql.org/message-id/15646-0d3caee9f6266f60%40postgresql.org

From my user point of view. if the boot value before transaction was
NULL then after transaction it MUST stay NULL and not empty string.

This will be consistent with that if I open new parallel session it
will return NULL.

Also I see NULL value (if setting was not defined) is useful.

Also this:
set_config( 'my.name', NULL )
must set 'my.name' to NULL or must issue warning about NULL is turned
into empty string.

Personally I dislike that NULL is converted to empty string. This
looks ugly. =(

--
Best regards,
Eugen Konkov

In response to

Responses

Browse pgsql-docs by date

  From Date Subject
Next Message PG Doc comments form 2019-02-22 11:03:43 Postgres make uninstall
Previous Message PG Doc comments form 2019-02-22 07:50:03 HEADER [ boolean ] not allowed in COPY ... FROM statement