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

2022-11-24 Thread zxwsbg
I don't know what happened, but an image seems to be better. -- Original -- From: "zxwsbg"

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

2022-11-24 Thread 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);

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

2022-11-23 Thread 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