From: Tomas Vondra <tomas.von...@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 we don't put the if block in the for loop yet. Regards Takayuki Tsunakawa
