On 2020-02-06 21:29:58 +0100, Peter Eisentraut wrote:
> On 2020-01-15 19:11, Andres Freund wrote:
> > /*
> > * Set up constraint/default info
> > */
> > if (has_not_null || ndef > 0 ||
> > attrmiss || relation->rd_rel->relchecks)
> > test, i.e. there are no defaults.
>
> > It does still strike me as not great that we can get a different
> > relcache entry, even if transient, depending on whether there are other
> > reasons to create a TupleConstr. Say a NOT NULL column.
> >
> > I'm inclined to think we should just also check has_generated_stored in
> > the if quoted above?
>
> Fixed that way.
Thanks.