Here's a rebase up to HEAD, mostly to placate the cfbot. I accounted for d8e34fa7a (s/all_baserels/all_query_rels/ in those places) and made one tiny bug-fix change. Nothing substantive as yet.
regards, tom lane
Hi,
For v3-0003-label-Var-nullability-in-parser.patch :
+ if (rtindex > 0 && rtindex <= list_length(pstate->p_nullingrels)) + relids = (Bitmapset *) list_nth(pstate->p_nullingrels, rtindex - 1); + else + relids = NULL; + + /* + * Merge with any already-declared nulling rels. (Typically there won't + * be any, but let's get it right if there are.) + */
+ if (relids != NULL)
It seems the last if block can be merged into the previous if block. That way `relids = NULL` can be omitted.