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


Reply via email to