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.
