David Rowley <dgrowle...@gmail.com> writes: > If there's no way to tell that the target entry comes from a left join, > then would it be a bit too quirky to just do the NOT NULL checking when > list_length(query->rtable) == 1 ? or maybe even loop over query->rtable and > abort if we find an outer join type? it seems a bit hackish, but perhaps it > would please more people than it would surprise.
Why do you think you can't tell if the column is coming from the wrong side of a left join? I don't recall right now if there is already machinery in the planner for this, but we could certainly deconstruct the from-clause enough to tell that. It's not hard to imagine a little function that recursively descends the join tree, not bothering to descend into the nullable sides of outer joins, and returns "true" if it finds a RangeTableRef for the desired RTE. If it doesn't find the RTE in the not-nullable parts of the FROM clause, then presumably it's nullable. The only thing wrong with that approach is if you have to call the function many times, in which case discovering the information from scratch each time could get expensive. I believe deconstruct_jointree already keeps track of whether it's underneath an outer join, so if we were doing this later than that, I'd advocate making sure it saves the needed information. But I suppose you're trying to do this before that. It might be that you could easily save aside similar information during the earlier steps in prepjointree.c. (Sorry for not having examined the patch yet, else I'd probably have a more concrete suggestion.) regards, tom lane -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers