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

Attachment: 0001-Disallow-sending-SI-messages-when-excuting-ROLLBACK.patch
Description: 0001-Disallow-sending-SI-messages-when-excuting-ROLLBACK.patch

Reply via email to