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