Hi, tom > >Hmmm, yeah, I think you're right. It probably doesn't make a big difference > >in > the real world --- anyone who's dependent on the performance of 2PC rollbaxks > is Doing It Wrong. > > But we'd have already done LocalExecuteInvalidationMessage when getting > out of the prepared transaction, so no other SI invals should be needed. > Yes, it does not make any error. > > But for the beginner, when understanding the code, it may make confused. > And for the developer, when developing based on this code, it may make > unnecessary handling added. > > So, I think it is better to optimize the code. > > Here is the patch. There was a problem with the before patch when testing. So resubmit it.
Regards, Liu Huailing
0001-Disallow-sending-SI-messages-when-excuting-ROLLBACK.patch
Description: 0001-Disallow-sending-SI-messages-when-excuting-ROLLBACK.patch