Re: BUG #17606: There is still some glitch in 3f7323cbb fixing failure of MULTIEXPR_SUBLINK

> After contemplating that for awhile, by far the least painful way

> seems to be to add the subLinkId to struct SubPlan.  We can get
> away with that in the back branches as long as we add it at the
> end, since SubPlans don't get recorded in the catalogs.

But let's assume that we manage to verify initplan/subplan by adding

subLinkId to SubPlan, we still have problem. In short, the exact

subqueryid of the new MULTIEXPR Param will be pain in the ass...

Consider the UPDATE sql containing sublinks by order like (subplan,

initplan, subplan, ...)

The existing code "offset = list_length(root->multiexpr_params)"

will count initplan into "offset", which is correct when first time here,

but it gives wrong subqueryid after...(when the place of initplan should

be "skipped")