My co-worker, Christine Poerschke, pointed out that the "completeList"
term was added in a change described as "restore old deletes via tlog
so peersync won't reorder".
If the goal was only the replay of deletes older than ourLowThreshold,
then keeping that goal doesn't need to interfere with the
Does "!completeList" do anything necessary in the line:
if (!completeList && Math.abs(otherVersion) < ourLowThreshold) break;
I think the line should simply be:
if (Math.abs(otherVersion) < ourLowThreshold) break;
-
The inclusion of "!completeList" in this conditional would seem to
only cau
