Re: Generate pg_stat_get_* functions with Macros - Mailing list pgsql-hackers

From Drouvot, Bertrand
Subject Re: Generate pg_stat_get_* functions with Macros
Date
Msg-id 292fc2a0-189f-c44e-1da6-6b4d71effdc2@gmail.com
Whole thread Raw
In response to Re: Generate pg_stat_get_* functions with Macros  (Michael Paquier <michael@paquier.xyz>)
Responses Re: Generate pg_stat_get_* functions with Macros  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
Hi,

On 12/5/22 8:44 AM, Michael Paquier wrote:
> On Mon, Dec 05, 2022 at 08:27:15AM +0100, Drouvot, Bertrand wrote:
>> On 12/4/22 6:32 PM, Nathan Bossart wrote:
>>> Alright.  I marked this as ready-for-committer.
>>
>> Thanks!
> 
> Well, that's kind of nice:
>   5 files changed, 139 insertions(+), 396 deletions(-)
> And I like removing code, so..
> 

Thanks for looking at it!

> In the same area, I am counting a total of 21 (?) pgstat routines for
> databases that rely on pgstat_fetch_stat_dbentry() while returning an
> int64.  This would lead to more cleanup.
> --


Yeah, good point, thanks!

I'll look at the "databases" ones but I think in a separate patch. The reason is that the current one is preparatory
workfor [1].
 
Means, once the current patch is committed, working on [1] and "cleaning" the databases one could be done in parallel.
Soundsgood to you?
 


[1]: https://commitfest.postgresql.org/41/3984/

Regards,

-- 
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: pg_dump: Remove "blob" terminology
Next
From: Dmitry Koval
Date:
Subject: Re: Operation log for major operations