Le mar. 8 oct. 2024 à 14:03, Benoit Lobréau <benoit.lobr...@dalibo.com> a
écrit :
> On 10/7/24 10:19, Guillaume Lelarge wrote:
> > I've done the split, but I didn't go any further than that.
>
> Thank you Guillaume. I have done the rest of the reformatting
> suggested by Michael but I decided to see If I have similar stuff
> in my logging patch and refactor accordingly if needed before posting
> the result here.
>
> I have hopes to finish it this week.
>
>
FWIW, with the recent commits of the pg_stat_statements patch, you need a
slight change in the patch I sent on this thread. You'll find a patch
attached to do that. You need to apply it after a rebase to master.
--
Guillaume.
diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c
index e749cdaa17..60a643b08d 100644
--- a/src/backend/executor/execMain.c
+++ b/src/backend/executor/execMain.c
@@ -484,10 +484,10 @@ standard_ExecutorEnd(QueryDesc *queryDesc)
Assert(estate != NULL);
- if (estate->es_parallelized_workers_planned > 0) {
+ if (estate->es_parallel_workers_to_launch > 0) {
pgstat_update_parallel_workers_stats(
- (PgStat_Counter) estate->es_parallelized_workers_planned,
- (PgStat_Counter) estate->es_parallelized_workers_launched);
+ (PgStat_Counter) estate->es_parallel_workers_to_launch,
+ (PgStat_Counter) estate->es_parallel_workers_launched);
}
/*
