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
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 |
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 |