pgsql: Check for two_phase change at end of process_syncing_tables_for_

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Check for two_phase change at end of process_syncing_tables_for_
Date: 2023-01-06 22:28:11
Message-ID: E1pDvC7-002OGr-H4@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Check for two_phase change at end of process_syncing_tables_for_apply.

Previously this function checked to see if we were ready to switch
to two_phase mode at its start, but that's silly: we should check
at the end, after we've done the work that might make us ready.
This simple change removes one sleep cycle from the time needed to
switch to two_phase mode. In the real world that might not be
worth much, but it shaves a few seconds off the runtime for the
subscription test suite.

Nathan Bossart

Discussion: https://postgr.es/m/20221122004119.GA132961@nathanxps13

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/4c032dd8046b145a25032643f536aab83deb19e3

Modified Files
--------------
src/backend/replication/logical/tablesync.c | 51 ++++++++++++++++-------------
1 file changed, 29 insertions(+), 22 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message David Rowley 2023-01-07 02:25:15 pgsql: Add additional regression tests for select_active_windows
Previous Message Andrew Dunstan 2023-01-06 21:06:02 pgsql: Allow hyphens in ltree labels