On Wed, Jul 16, 2025 at 9:17 AM Mircea Cadariu <cadariu.mir...@gmail.com> wrote:
> Thanks for the elaboration and updated patch! Indeed, I see it's set in
> the ScanKeyEntryInitialize to either BTGreaterStrategyNumber or
> BTLessStrategyNumber, then few lines lower there's the if with the break.
>
> I'm convinced.

Pushed.

> I noticed this CI job failure for the V2, seems unrelated to the subject
> of the patch though, does it need a retry?
> https://cirrus-ci.com/task/5781246762024960

It must be unrelated. I didn't see any such failure myself just now,
and I can't imagine how it could possibly be relevant.

Thanks for the review
-- 
Peter Geoghegan


Reply via email to