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

[zxwsbg]

Hi, guys:


In PG14 or higher version, I notice that SnapBuildRestore don't 
set builder->next_phase_at=InvalidTransactionId .


But in SnapBuildSerialize function, the assert check this condition.


Assert(builder->next_phase_at == InvalidTransactionId);



I'd like to know is this right?


Thanks.

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

[zxwsbg]

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

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"