From: Michael P. <mic...@gm...> - 2014-01-09 05:44:08
|
On Thu, Jan 9, 2014 at 2:33 PM, Koichi Suzuki <koi...@gm...> wrote: > Thanks a lot for the analysis and the patch. > > The patch is not for XC-specific code. Do you think the patch should > go to PG as well? No. The code path taken in the first section is exclusive to XC: it uses an internal 2PC. IMO the patch is fine as I recall (some souvenir, feel free to correct me) that in this code path a transaction ID is already assigned, so GetTopTransactionId is not going to call GTM. Regards, -- Michael |