Re: Fix --missing-stats-only false positive for partitioned expression indexes

[Corey Huinker]

On Tue, Jun 16, 2026 at 12:14 PM Nathan Bossart <nathandboss...@gmail.com>
wrote:

> On Tue, Jun 16, 2026 at 10:25:52AM -0500, Baji Shaik wrote:
> > I tested "vacuumdb --missing-stats-only"  and found that it flags
> > partitioned tables that have expression indexes, even after a full
> > ANALYZE.
>
> Thanks for reporting.
>
> -                                                        " OR EXISTS
> (SELECT NULL FROM pg_catalog.pg_attribute a\n"
> +                                                        " OR (NOT
> p.inherited"
> +                                                        " AND EXISTS
> (SELECT NULL FROM pg_catalog.pg_attribute a\n"
>
> I'm curious why you added this check to the beginning and surrounded the
> rest with parentheses.  Wouldn't it be better to follow the example of the
> surrounding clauses and an "AND NOT p.inherited" somewhere in the middle?
>
> --
> nathan


I had a similar question and am toying around with refactoring it now.

The rest of the patch checks out aside from that.