Re:Why not set builder->next_phase_at=InvalidTransactionId in SnapBuildRestore function?

[zxwsbg]

I don't know what happened, but an image seems to be better.
------------------ Original ------------------
From:                                                                           
                                             "zxwsbg"                           
                                                         <zxw...@qq.com&gt;;
Date:&nbsp;Thu, Nov 24, 2022 04:01 PM
To:&nbsp;"pgsql-general"<pgsql-gene...@postgresql.org&gt;;

Subject:&nbsp;Why not set builder-&gt;next_phase_at=InvalidTransactionId in 
SnapBuildRestore function?




Last email has some problem, I resend one.
 
In PG14 or higher version, I notice that SnapBuildRestore function don't set 
builder-&gt;next_phase_at=InvalidTransactionId .
 
But in SnapBuildSerialize function, the assert check this condition.
 
Assert(builder-&gt;next_phase_at == InvalidTransactionId);
 
I'd like to know is this right?
 
Thanks.