Make a 100_bugs.pl test more faster.

Started by Anton A. Melnikovover 3 years ago2 messageshackers
Jump to latest
#1Anton A. Melnikov
aamelnikov@inbox.ru

Hello!

The previous discussion was here:
/messages/by-id/b570c367-ba38-95f3-f62d-5f59b9808226@inbox.ru

On 15.11.2022 04:59, Tom Lane wrote:

"Anton A. Melnikov" <aamelnikov@inbox.ru> writes:

Additionally
i've tried to reduce overall number of nodes previously
used in this test in a similar way.

Optimizing existing tests isn't an answer to that. We could
install those optimizations without adding a new test case.

Here is a separate patch for the node usage optimization mentioned above.
It decreases the CPU usage during 100_bugs.pl by about 30%.

There are also some experimental data: 100_bugs-CPU-usage.txt

Would be glad for any comments and concerns.

With best regards,

--
Anton A. Melnikov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company

Attachments:

v1-0001-Reuse-nodes-in-100_bugs-test.patchtext/x-patch; charset=UTF-8; name=v1-0001-Reuse-nodes-in-100_bugs-test.patchDownload+8-9
100_bugs-CPU-usage.txttext/plain; charset=UTF-8; name=100_bugs-CPU-usage.txtDownload
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Anton A. Melnikov (#1)
Re: Make a 100_bugs.pl test more faster.

"Anton A. Melnikov" <aamelnikov@inbox.ru> writes:

Here is a separate patch for the node usage optimization mentioned above.
It decreases the CPU usage during 100_bugs.pl by about 30%.

Hmm ... as written, this isn't testing the same thing, because you
didn't disable the FOR ALL TABLES publications created in the earlier
steps, so we're redundantly syncing more publications in the later
ones. Cleaning those up seems to make it a little faster yet,
so pushed with that adjustment.

regards, tom lane