Pavel Stehule wrote: > I fixed two issues. > > 2. there was reverse setting in NOT NULL flag
Ah-hah, that was silly, thanks. > 1. there are not columns data when there are not any explicit column - fixed Hmm. Now that I see how this works, by having the GetValue "guess" what is going on and have a special case for it, I actually don't like it very much. It seems way too magical. I think we should do away with the "if column is NULL" case in GetValue, and instead inject a column during transformTableExpr if columns is NIL. This has implications on ExecInitExpr too, which currently checks for an empty column list -- it would no longer have to do so. Maybe this means we need an additional method, which would request "the expr that returns the whole row", so that transformExpr can work for XmlTable (which I think would be something like "./") and the future JsonTable stuff (I don't know how that one would work, but I assume it's not necessarily the same thing). -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
