From dcfecaae9e35afe4fb11304267f547141842447d Mon Sep 17 00:00:00 2001 From: Robert Haas Date: Tue, 8 Mar 2016 10:27:03 -0500 Subject: [PATCH] Fix parallel query on standby servers. Without this fix, it inevitably bombs out with "ERROR: failed to initialize transaction_read_only to 0". Repair. Ashutosh Sharma; comments adjusted by me. --- src/backend/commands/variable.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/backend/commands/variable.c b/src/backend/commands/variable.c index 903b3a6313..f801faacd2 100644 --- a/src/backend/commands/variable.c +++ b/src/backend/commands/variable.c @@ -482,11 +482,13 @@ show_log_timezone(void) * nothing since XactReadOnly will be reset by the next StartTransaction(). * The IsTransactionState() test protects us against trying to check * RecoveryInProgress() in contexts where shared memory is not accessible. + * (Similarly, if we're restoring state in a parallel worker, just allow + * the change.) */ bool check_transaction_read_only(bool *newval, void **extra, GucSource source) { - if (*newval == false && XactReadOnly && IsTransactionState()) + if (*newval == false && XactReadOnly && IsTransactionState() && !InitializingParallelWorker) { /* Can't go to r/w mode inside a r/o transaction */ if (IsSubTransaction()) -- 2.40.0