From: Tomas Vondra <tomas.vondra@2ndquadrant.com>
To: Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>
Cc: alvherre@2ndquadrant.com
Cc: andres@anarazel.de
Cc: ah@cybertec.at
Cc: magnus@hagander.net
Cc: robertmhaas@gmail.com
Cc: tgl@sss.pgh.pa.us
Cc: pgsql-hackers@postgresql.org
Subject: Re: shared-memory based stats collector
Date: Wed, 28 Nov 2018 23:13:53 +0100
Message-ID: <2e77189a-48f0-ea50-c0b7-507d22d37032@2ndquadrant.com> (raw)
In-Reply-To: <20181127.175949.06807946.horiguchi.kyotaro@lab.ntt.co.jp>
References: <20181112.201042.147595779.horiguchi.kyotaro@lab.ntt.co.jp>
<8e041a34a70ae5d44cb10a7e516308e2f6d1b651.camel@2ndquadrant.com>
<6c079a69-feba-e47c-7b85-8a9ff31adef3@2ndquadrant.com>
<20181127.175949.06807946.horiguchi.kyotaro@lab.ntt.co.jp>
On 11/27/18 9:59 AM, Kyotaro HORIGUCHI wrote:
> ...
>
> v10-0001-sequential-scan-for-dshash.patch
> v10-0002-Add-conditional-lock-feature-to-dshash.patch
> fixed.
> v10-0003-Make-archiver-process-an-auxiliary-process.patch
> fixed.
> v10-0004-Shared-memory-based-stats-collector.patch
> updated not to touch guc.
> v10-0005-Remove-the-GUC-stats_temp_directory.patch
> collected all guc-related changes.
> updated not to break other programs.
> v10-0006-Split-out-backend-status-monitor-part-from-pgstat.patch
> basebackup.c requires both bestats.h and pgstat.h
> v10-0007-Documentation-update.patch
> small change related to 0005.
>
I need to do a more thorough review of part 0006, but these patches
seems quite fine to me. I'd however merge 0007 into the other relevant
parts (it seems like a mix of docs changes for 0004, 0005 and 0006).
Thinking about it a bit more, I'm wondering if we need to keep 0004 and
0005 separate. My understanding is that the stats_temp_directory is used
only from the stats collector, so it probably does not make much sense
to keep it after 0004. We may also keep it separate and then commit both
0004 and 0005 together, of course. What do you think.
regards
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: tomas.vondra@2ndquadrant.com, horiguchi.kyotaro@lab.ntt.co.jp, alvherre@2ndquadrant.com, andres@anarazel.de, ah@cybertec.at, magnus@hagander.net, robertmhaas@gmail.com, tgl@sss.pgh.pa.us
Subject: Re: shared-memory based stats collector
In-Reply-To: <2e77189a-48f0-ea50-c0b7-507d22d37032@2ndquadrant.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox