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