On Sat, Oct 18, 2025 at 9:16 AM Chao Li <[email protected]> wrote: > I think I put all concentration on the big picture yesterday, so I ignored > this implementation detail: > > ``` > + if (lag_tracker->overflowed[head].lsn > lsn) > + return now - lag_tracker->overflowed[head].time; > ``` > > Should this “>” be “>=“?
I think either ">" or ">=" would work. However, the following loop in LagTrackerRead() advances the read head when the LSN in the current buffer entry is equal to the given LSN, so I followed the same logic for the overflow entry and used ">" there. Regards, -- Fujii Masao
