Hi Ilia, thanks for the review! I've fixed the cosmetic nits in v7 and attached it to the next email.
Best regards, Enrique. On, 24 Jul 2026 at 17:03, Ilia Evdokimov (<[email protected]>) wrote: > The overall approach looks architecturally correctly. Just two cosmetic > nits: > > 1. In `mcv_can_cap()`, the early-exit branch uses braces around `return > false`. This would normally be written without braces. > > 2. The doc comment above `mcv_can_cap()` says condition requires the > clause to be either an equality operator, or a bare boolean Var, but > implementation actually handles one more cases - `NOT var` > > 3. Apply pgindent into diff.
