On 12/12/21 14:47, Zhihong Yu wrote:


On Sat, Dec 11, 2021 at 9:14 PM Tomas Vondra <tomas.von...@enterprisedb.com <mailto:tomas.von...@enterprisedb.com>> wrote:
>
...

     > +       /* skip statistics with mismatching stxdinherit value */
     > +       if (stat->inherit != rte->inh)
     >
     > Should a log be added for the above case ?
     >

    Why should we log this? It's an entirely expected case - there's a
    mismatch between inheritance for the relation and statistics, simply
    skipping it is the right thing to do.


Hi,
I agree that skipping should be fine (to avoid too much logging).


I'm not sure it's related to the amount of logging, really. It'd be just noise without any practical use, even for debugging purposes. If you have an inheritance tree, it'll automatically have one set of statistics for inh=true and one for inh=false. And this condition will always skip one of those, depending on what query is being estimated.


regards

--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Reply via email to