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

I don't know what happened, but an image seems to be better.
------------------ Original ------------------
From: "zxwsbg" <zxwsbg(at)qq(dot)com&gt;;
Date:&nbsp;Thu, Nov 24, 2022 04:01 PM
To:&nbsp;"pgsql-general"<pgsql-general(at)postgresql(dot)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.