Dear Sawada-san, > Thank you for the comments. I'm now convinced that we can call it > after resetting the top-level XID. I was concerned about the > inconsistency between the AtEOXact_LogicalCtl() calls in > CommitTransaction() and CleanupTransaction(), but it's fine for them > to differ, since we're holding interrupts in the former but not in the > latter. Calling AtEOXact_LogicalCtl() after resetting the top-level > XID would be more bullet-proof, so let's go with this approach. > > I've attached the updated patch. Please review it.
Understood, LGTM. Best regards, Hayato Kuroda FUJITSU LIMITED
