Disable distributed commit during proc_exit_inprogress.

Author: arssher

src/commit.c

@@ -188,7 +188,13 @@ MtmBeginTransaction()
 	/* XXX: clean MtmTx on commit and check on begin that it is clean. */
 	/* That should unveil probable issues with subxacts. */
 
-	if (!MtmIsEnabled())
+	/*
+	 * XXX: proc_exit_inprogress added after weird failure in JIT guts
+	 * (through MtmLoadConfig) on RemoveTempRelationsCallback xact start. The
+	 * reasons are not exactly clear, but we hardly need distributed xacts
+	 * during shutdown.
+	 */
+	if (proc_exit_inprogress || !MtmIsEnabled())
 	{
 		MtmTx.distributed = false;
 		return;

src/multimaster.c

@@ -1444,7 +1444,7 @@ mtm_join_node(PG_FUNCTION_ARGS)
 }
 
 /*
- * Check that all current gen members have the same gen and doesn't want to
+ * Check that all current gen members have the same gen and don't want to
  * change it. This is useful for learning when cluster has finally started up
  * and no generation switch perturbations will happen anymore unless some
  * failure happens. Note that it asserts that all gen members are online -- we

src/pglogical_apply.c

@@ -197,6 +197,7 @@ process_remote_begin(StringInfo s, MtmReceiverWorkerContext *rwctx)
 	int sender_node_id = pq_getmsgint(s, 4);
 	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);
 
 	InterruptPending = false;
 	QueryCancelPending = false;