On 11/8/2025 20:15, Greg Sabino Mullane wrote:
> I'm not convinced this is an improvement from someone just coming in to
> this part of the code, especially given (for example) the comment right
> above it:
>
> * Determine if the inner table can duplicate outer rows. We must
> * bypass the unique rel cache here since we're possibly using aThanks for your feedback.
I made some minor adjustments to the comments to make the code more
consistent. Sure, rrel and krel don't seem like the best solution - I
guess natives could find less wordy synonyms than just dumb
'keeping_rel' and 'removing_rel'. But it is simpler to track which
relation and RowMark should be removed.
--
regards, Andrei Lepikhov