On 18/12/2025 11:56, Jasper Smit wrote:
> The test is really nice with the injection points and the dynamically
> sized data.
>
>> Ah, but this codepath is taken when HEAP_KEYS_UPDATED is *not* set. I
>> got that backwards. So I agree the ItemPointerEquals(&tuple->t_self,
>> ctid) check is redundant.
>
> Ok, I did not think about deletes. So the boolean updated here could
> mean both update and delete, that makes sense to me.
Committed and backpatched this now. Thanks!
- Heikki