From: | Melanie Plageman <melanieplageman(at)gmail(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Fabrízio de Royes Mello <fabriziomello(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>, Andres Freund <andres(at)anarazel(dot)de>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> |
Subject: | Re: Remove io prefix from pg_stat_io columns |
Date: | 2023-04-20 00:50:13 |
Message-ID: | CAAKRu_bo3QPWRgoLa30PHb0CH3KnbPFSSncBMksdOjngk+5AWQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Apr 19, 2023 at 8:42 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Wed, Apr 19, 2023 at 01:54:21PM -0300, Fabrízio de Royes Mello wrote:
> > On Wed, Apr 19, 2023 at 1:27 PM Melanie Plageman <melanieplageman(at)gmail(dot)com> wrote:
> >> Over in [1], we discussed removing the "io" prefix from the columns
> >> "io_context" and "io_object" in pg_stat_io since they seem redundant
> >> given the view name
> >
> > LGTM. All tests passed and were built without warnings.
>
> There are a lot of internal references to both of them mainly around
> the buffer manager and the pgstat code, still I agree that the view
> feels redundant as currently written, so agreed. It does not seem
> like you have missed any references here, from what I can see.
I thought about changing parameter and local variable names to remove
the prefix, but in the original discussion folks seemed to think it made
sense to leave the "C level" references with an "io" prefix. I think we
could change many of them, but some of them may be required for clarity.
- Melanie
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2023-04-20 01:09:17 | Re: Autogenerate some wait events code and documentation |
Previous Message | Michael Paquier | 2023-04-20 00:48:08 | Re: Autogenerate some wait events code and documentation |