Check for two_phase change at end of process_syncing_tables_for_apply.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 6 Jan 2023 20:35:31 +0000 (15:35 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 6 Jan 2023 22:27:58 +0000 (17:27 -0500)
commit4c032dd8046b145a25032643f536aab83deb19e3
tree49bd7a23e14d6e3a8a1ec07aa8b8f145c2edf626
parentb1665bf01e5f4200d37addfc2ddc406ff7df14a5
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
src/backend/replication/logical/tablesync.c