From: Karthik Nayak <karthik.188@gmail.com>
Date: Wed, 19 Feb 2025 20:13:48 GMT
Subject: Re: [PATCH 10/14] refs/iterator: implement seeking for reftable iterators
Message-ID: <CAOLa=ZSgpeTCayxCs01hdVJ3FsXMPCzSg+RV5tgMc0Agz9nAqg@mail.gmail.com>
In-Reply-To: <20250217-pks-update-ref-optimization-v1-10-a2b6d87a24af@pks.im>

Patrick Steinhardt <ps@pks.im> writes:

> Implement seeking of reftable iterators. As the low-level reftable
> iterators already support seeking this change is straight-forward. Two
> notes though:
>
>   - We do not support seeking on reflog iterators.
>

Nit: this doesn't explain the reason and it would be nice to state why
we do not support reflog iterators here

>   - We start to check whether `reftable_stack_init_ref_iterator()` is
>     successful.
>

The patch looks good!

[snip]