Re: [COMMITTERS] pgsql: Bloom index contrib module

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Teodor Sigaev <teodor(at)sigaev(dot)ru>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Bloom index contrib module
Date: 2016-04-09 16:46:38
Message-ID: 14783.1460220398@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

I wrote:
> Would it be possible to dial down the amount of runtime consumed by
> the regression tests for this module?

Hmm ... "perf" says that a full 50% of the runtime of contrib/bloom's
regression test is consumed by GenericXLogFinish, and of that, the vast
majority is consumed by the extremely inefficient one-byte-at-a-time
matching in writeDelta() --- even dwarfing the "exchange two page images"
memcpy's. So there seems to be good reason to expend some
micro-optimization effort in that area. But aside from the writeDelta
problems, what in the world is the reason for the page exchange logic?
I could see copying a working image into place in the shared buffer
arena, but not saving the previous contents.

Even if these costs went to zero, though, the bloom regression tests
would still be consuming a disproportionate amount of time.

regards, tom lane

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2016-04-09 19:02:39 pgsql: Code review/prettification for generic_xlog.c.
Previous Message Tom Lane 2016-04-09 15:50:08 Re: [COMMITTERS] pgsql: Bloom index contrib module

Browse pgsql-hackers by date

  From Date Subject
Next Message Tatsuo Ishii 2016-04-09 17:37:57 Re: multivariate statistics v14
Previous Message Daniel Verite 2016-04-09 15:50:08 Re: [patch] Proposal for \crosstabview in psql