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