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