Re: Direct I/O

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Justin Pryzby <pryzby(at)telsasoft(dot)com>, Noah Misch <noah(at)leadboat(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org>, Christoph Berg <myon(at)debian(dot)org>, mikael(dot)kjellstrom(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Direct I/O
Date: 2023-08-22 12:15:34
Message-ID: 2769341e-fa28-c2ee-3e4b-53fdcaaf2271@eisentraut.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 01.05.23 04:47, Thomas Munro wrote:
> On Mon, May 1, 2023 at 12:00 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Justin Pryzby <pryzby(at)telsasoft(dot)com> writes:
>>> On Sun, Apr 30, 2023 at 06:35:30PM +1200, Thomas Munro wrote:
>>>> What about a
>>>> warning message about that at startup if it's on?
>>
>>> Such a warning wouldn't be particularly likely to be seen by someone who
>>> already didn't read/understand the docs for the not-feature that they
>>> turned on.
>>
>> Yeah, I doubt that that would be helpful at all.
>
> Fair enough.
>
>>> Since this is -currently- a developer-only feature, it seems reasonable
>>> to rename the GUC to debug_direct_io, and (at such time as it's
>>> considered to be helpful to users) later rename it to direct_io.
>>
>> +1
>
> Yeah, the future cross-version confusion thing is compelling. OK,
> here's a rename patch. I left all the variable names and related
> symbols as they were, so it's just the GUC that gains the prefix. I
> moved the documentation hunk up to be sorted alphabetically like
> nearby entries, because that seemed to look nicer, even though the
> list isn't globally sorted.

I suggest to also rename the hook functions (check and assign), like in
the attached patch. Mainly because utils/guc_hooks.h says to order the
functions by GUC variable name, which was already wrong under the old
name, but it would be pretty confusing to sort the functions by their
GUC name that doesn't match the function names.

Attachment Content-Type Size
0001-Rename-hook-functions-for-debug_io_direct-to-match-v.patch text/plain 3.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Chapman Flack 2023-08-22 12:16:02 Re: Extract numeric filed in JSONB more effectively
Previous Message Vik Fearing 2023-08-22 12:01:19 Re: list of acknowledgments for PG16