On Sat, Oct 18, 2025 at 9:16 AM Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> 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