Peter Eisentraut <peter.eisentr...@enterprisedb.com> writes: > On 04.03.21 20:01, Tom Lane wrote: >> (2) Does that datatype change need to propagate anywhere besides >> what I touched here? I did not make any effort to search for >> other places.
> I think > Var.varnosyn > CurrentOfExpr.cvarno > should also have their type changed. Agreed as to CurrentOfExpr.cvarno. But I think the entire point of varnosyn is that it saves the original rangetable reference and *doesn't* get overwritten with OUTER_VAR etc. So that one is a different animal, and I'm inclined to leave it as Index. regards, tom lane