Re: shared-memory based stats collector

From: Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>
To: andres(at)anarazel(dot)de
Cc: ah(at)cybertec(dot)at, magnus(at)hagander(dot)net, robertmhaas(at)gmail(dot)com, tgl(at)sss(dot)pgh(dot)pa(dot)us, pgsql-hackers(at)postgresql(dot)org
Subject: Re: shared-memory based stats collector
Date: 2018-10-02 07:06:51
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

The previous patch doesn't work...

At Thu, 27 Sep 2018 22:00:49 +0900 (Tokyo Standard Time), Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> wrote in <20180927(dot)220049(dot)168546206(dot)horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>
> - 0001 to 0006 is rebased version of v4.
> - 0007 adds conditional locking to dshash
> - 0008 is the no-UDP stats collector.
> If required lock is not acquired for some stats items, report
> funcions immediately return after storing the values locally. The
> stored values are merged with later calls. Explicitly calling
> pgstat_cleanup_pending_stat() at a convenient timing tries to
> apply the pending values, but the function is not called anywhere
> for now.
> stats collector process is used only to save and load saved stats
> files and create shared memory for stats. I'm going to remove
> stats collector.
> I'll continue working this way.

It doesn't work nor even compile since I failed to include some
changes. The atached v6-0008 at least compiles and words.

0001-0007 are not attached since they are still aplicable on
master head with offsets.


Kyotaro Horiguchi
NTT Open Source Software Center

Attachment Content-Type Size
v6-0008-Ultra-PoC-of-full-shared-memory-stats-collector.patch text/x-patch 96.8 KB

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Langote 2018-10-02 07:11:54 Re: Expression errors with "FOR UPDATE" and postgres_fdw with partition wise join enabled.
Previous Message John Naylor 2018-10-02 07:06:48 Re: inconsistency and inefficiency in setup_conversion()