On Wed, Nov 24, 2021 at 10:44 PM Masahiko Sawada <sawada.m...@gmail.com> wrote: > > I've attached an updated version patch. Unless I miss something, all > comments I got so far have been incorporated into this patch. Please > review it. >
Only a couple of minor points: src/backend/postmaster/pgstat.c (1) pgstat_get_subworker_entry In the following comment, it should say "returns an entry ...": + * apply worker otherwise returns entry of the table sync worker associated src/include/pgstat.h (2) typedef struct PgStat_StatDBEntry "subworker" should be "subworkers" in the following comment, to match the struct member name: * subworker is the hash table of PgStat_StatSubWorkerEntry which stores Otherwise, the patch LGTM. Regards, Greg Nancarrow Fujitsu Australia
