On Tue, Jan 7, 2025 at 7:58 AM Matthias van de Meent <[email protected]> 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
