On 2022-Jan-21, Japin Li wrote:
> + /*
> + * NOT MATCHED actions can't see target relation, but they can see
> + * source relation.
> + */
> + Assert(mergeWhenClause->commandType == CMD_INSERT ||
> + mergeWhenClause->commandType == CMD_DELETE ||
> + mergeWhenClause->commandType == CMD_NOTHING);
> + setNamespaceVisibilityForRTE(pstate->p_namespace,
> + targetRelRTE, false, false);
> + setNamespaceVisibilityForRTE(pstate->p_namespace,
> + sourceRelRTE, true, true);
>
> Should we remove the CMD_DELETE from Assert(), since it will not happened
> according to MERGE syntax?
Absolutely --- silly copy&paste mistake. Pushed fix.
--
Álvaro Herrera 39°49'30"S 73°17'W — https://www.EnterpriseDB.com/
"Ni aún el genio muy grande llegaría muy lejos
si tuviera que sacarlo todo de su propio interior" (Goethe)
