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