On Tue, Jan 7, 2025 at 7:58 AM Matthias van de Meent
<boekewurm+postgres@gmail.com> wrote:
> This doesn't really clarify _why_ we'd never get this far, so I'd word that as
>
> + * Cannot be a NULL in the first row member: _bt_preprocess_keys
> + * would've marked the qual as unsatisfyable, preventing us from
> + * ever getting this far.
>
> Apart from that minor issue, LGTM.
Pushed this just now. I used your suggested wording in the committed patch.
Thanks for the review!
--
Peter Geoghegan