Use sync commit for logical replication apply in TAP tests

From: Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com>
To: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Subject: Use sync commit for logical replication apply in TAP tests
Date: 2017-04-19 02:25:42
Message-ID: f8698d02-35c0-35f9-c60e-642ab84afda3@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

The commit 887227a1c changed the defaults for subscriptions to do async
commit. But since the tests often wait for disk flush and there is no
concurrent activity this has increased the amount of time needed for
each test. So the attached patch changes the subscriptions create in tab
tests to use sync commit which improves performance there (because we
also replicate only very few transactions in the tests).

--
Petr Jelinek http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

Attachment Content-Type Size
0001-Change-logical-replication-TAP-tests-to-use-sync-com.patch text/plain 5.8 KB

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2017-04-19 02:29:08 Re: Failed recovery with new faster 2PC code
Previous Message Petr Jelinek 2017-04-19 02:18:18 Re: some review comments on logical rep code