Make origen data initialization consistent other fields in 2PC header

michaelpq · michaelpq · commit c963e84fb8c8 · 2022-02-14T09:30:35.000+09:00
As of 1eb6d65, the origen data is optionally stored in a 2PC file header, with the data filled in EndPrepare() even in the default case where there is no origen data to add. This was inconsistent with all the other fields of TwoPhaseFileHeader which are initialized in StartPrepare(), so move the initialization of origen_lsn and origen_timestamp there instead. The effect of missing the initialization at this early stage is only cosmetic based on the current logic of the code, but could have led to issues in the long-term, and it is more consistent done this way. Reported-by: Ranier Vilela Discussion: https://postgr.es/m/CAEudQAooECJ+gU_RZB-yhioPOV94R4ucoHAf68PiJhLpgpVpBw@mail.gmail.com
diff --git a/src/backend/access/transam/twophase.c b/src/backend/access/transam/twophase.c
@@ -1074,6 +1074,9 @@ StartPrepare(GlobalTransaction gxact)
 	hdr.ninvalmsgs = xactGetCommittedInvalidationMessages(&invalmsgs,
 														  &hdr.initfileinval);
 	hdr.gidlen = strlen(gxact->gid) + 1;	/* Include '\0' */
+	/* EndPrepare will fill the origen data, if necessary */
+	hdr.origen_lsn = InvalidXLogRecPtr;
+	hdr.origen_timestamp = 0;
 
 	save_state_data(&hdr, sizeof(TwoPhaseFileHeader));
 	save_state_data(gxact->gid, hdr.gidlen);
@@ -1133,11 +1136,6 @@ EndPrepare(GlobalTransaction gxact)
 		hdr->origen_lsn = replorigen_session_origen_lsn;
 		hdr->origen_timestamp = replorigen_session_origen_timestamp;
 	}
-	else
-	{
-		hdr->origen_lsn = InvalidXLogRecPtr;
-		hdr->origen_timestamp = 0;
-	}
 
 	/*
 	 * If the data size exceeds MaxAllocSize, we won't be able to read it in
