From: Tomas Vondra <tomas.vondra@enterprisedb.com>
> Right. But I think Tom is right this should initialize ri_BatchSize for all the
> resultRelInfo elements, not just the first one. Per the attached patch, which
> resolves the issue both on x86_64 and armv7l for me.
I think Your patch is perfect in the sense that it's ready for the future multi-target DML support. +1
Just for learning, could anyone tell me what this loop for? I thought current Postgres's DML supports a single target
table,so it's enough to handle the first element of mtstate->resultRelInfo. In that sense, Amit-san and I agreed that
wedon't put the if block in the for loop yet.
Regards
Takayuki Tsunakawa