fix transaction parsing

In the commit 04e72dd the code for reading
sender_node_id and action type was removed. This commit fixes this issue.

Author: Mikhail Rutman

src/pglogical_apply.c

@@ -194,9 +194,10 @@ static void
 process_remote_begin(StringInfo s, MtmReceiverWorkerContext *rwctx)
 {
 	/* there is no need to send this, but since we do, check its sanity */
-#ifdef USE_ASSERT_CHECKING
 	int sender_node_id = pq_getmsgint(s, 4);
-#endif
+
+	(void) sender_node_id;		/* keep the compiler quiet when asserts are disabled*/
+
 	Assert(rwctx->sender_node_id == sender_node_id);
 	rwctx->origin_xid = pq_getmsgint64(s);
 	mtm_log(MtmApplyTrace, "processing begin of xid " XID_FMT, rwctx->origin_xid);

src/pglogical_receiver.c

@@ -318,12 +318,12 @@ MtmFilterTransaction(char *record, int size, Syncpoint *spvector,
 	}
 	else if (msgtype == 'M')
 	{
-#ifdef USE_ASSERT_CHECKING
 		char		action = pq_getmsgbyte(&s);
-#endif
 		int			messageSize;
 		char const *messageBody;
 
+		(void) action;		/* keep the compiler quiet when asserts are disabled*/
+
 		end_lsn = pq_getmsgint64(&s);
 		messageSize = pq_getmsgint(&s, 4);
 		messageBody = pq_getmsgbytes(&s, messageSize);