Re: Retire has_multiple_baserels()

On Tue, Oct 10, 2023 at 5:43 PM Aleksander Alekseev <
aleksander(at)timescale(dot)com> wrote:

> I used the following patch to double check that nothing was missed:
>
> ```
> --- a/src/backend/optimizer/path/allpaths.c
> +++ b/src/backend/optimizer/path/allpaths.c
> @@ -2207,8 +2207,13 @@ has_multiple_baserels(PlannerInfo *root)
> /* ignore RTEs that are "other rels" */
> if (brel->reloptkind == RELOPT_BASEREL)
> if (++num_base_rels > 1)
> + {
> +
> Assert(bms_membership(root->all_baserels) == BMS_MULTIPLE);
> return true;
> + }
> }
> +
> + Assert(bms_membership(root->all_baserels) != BMS_MULTIPLE);
> return false;
> }
> ```
>
> It wasn't. The patch LGTM.

Thanks for the verification.

Thanks
Richard