Set replication origin when decoding commit records.

anarazel · anarazel · commit f3a764b0dac7 · 2015-11-09T00:03:35.000+01:00
By accident the replication origin was not set properly in
DecodeCommit(). That's bad because the origin is passed to the output
plugins origin filter, and accessible from the output plugin via
ReorderBufferTXN-&gt;origin_id.  Accessing the origin of individual changes
worked before the fix, which is why this wasn't notices earlier.

Reported-By: Craig Ringer
Author: Craig Ringer
Discussion: CAMsr+YFhBJLp=qfSz3-J+0P1zLkE8zNXM2otycn20QRMx380gw@mail.gmail.com
Backpatch: 9.5, where replication origins where introduced
diff --git a/src/backend/replication/logical/decode.c b/src/backend/replication/logical/decode.c
@@ -450,7 +450,7 @@ DecodeCommit(LogicalDecodingContext *ctx, XLogRecordBuffer *buf,
 {
 	XLogRecPtr	origin_lsn = InvalidXLogRecPtr;
 	XLogRecPtr	commit_time = InvalidXLogRecPtr;
-	XLogRecPtr	origin_id = InvalidRepOriginId;
+	XLogRecPtr	origin_id = XLogRecGetOrigin(buf->record);
 	int			i;
 
 	if (parsed->xinfo & XACT_XINFO_HAS_ORIGIN)

Original file line number	Diff line number	Diff line change
`@@ -450,7 +450,7 @@ DecodeCommit(LogicalDecodingContext ctx, XLogRecordBuffer buf,`
`450`	`450`	`{`
`451`	`451`	`XLogRecPtr origin_lsn = InvalidXLogRecPtr;`
`452`	`452`	`XLogRecPtr commit_time = InvalidXLogRecPtr;`
`453`		`- XLogRecPtr origin_id = InvalidRepOriginId;`
	`453`	`+ XLogRecPtr origin_id = XLogRecGetOrigin(buf->record);`
`454`	`454`	`int i;`
`455`	`455`
`456`	`456`	`if (parsed->xinfo & XACT_XINFO_HAS_ORIGIN)`