On 2019/04/19 14:39, Etsuro Fujita wrote:
> (2019/04/19 13:00), Amit Langote wrote:
>> On 2019/04/18 22:10, Etsuro Fujita wrote:
>>> * I kept all the changes in the previous patch, because otherwise
>>> postgres_fdw would fail to release resources for a foreign-insert
>>> operation created by postgresBeginForeignInsert() for a tuple-routable
>>> foreign table (ie, a foreign-table subplan resultrel that has been updated
>>> already) during postgresEndForeignInsert().
>>
>> Hmm are you saying that the cases for which we'll still allow tuple
>> routing (foreign table receiving moved-in rows has already been updated),
>> there will be two fmstates to be released -- the original fmstate
>> (UPDATE's) and aux_fmstate (INSERT's)?
>
> Yeah, but I noticed that that explanation was not correct. (I think I was
> really in hurry.) See the correction in [1].
Ah, I hadn't noticed your corrected description in [1] even though your
message was in my inbox before I sent my email.
Thanks,
Amit