From 987277f402a31ba91ea647d115785e5a4f7850fa Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Wed, 5 Sep 2012 21:49:18 -0400 Subject: [PATCH] Fix inappropriate error messages for Hot Standby misconfiguration errors. Give the correct name of the GUC parameter being complained of. Also, emit a more suitable SQLSTATE (INVALID_PARAMETER_VALUE, not the default INTERNAL_ERROR). Gurjeet Singh, errcode adjustment by me --- src/backend/access/transam/xlog.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c index 9a9b04f242..bdbeb84095 100644 --- a/src/backend/access/transam/xlog.c +++ b/src/backend/access/transam/xlog.c @@ -5645,9 +5645,10 @@ GetXLogReceiptTime(TimestampTz *rtime, bool *fromStream) */ #define RecoveryRequiresIntParameter(param_name, currValue, minValue) \ do { \ - if (currValue < minValue) \ + if ((currValue) < (minValue)) \ ereport(ERROR, \ - (errmsg("hot standby is not possible because " \ + (errcode(ERRCODE_INVALID_PARAMETER_VALUE), \ + errmsg("hot standby is not possible because " \ "%s = %d is a lower setting than on the master server " \ "(its value was %d)", \ param_name, \ @@ -5688,10 +5689,10 @@ CheckRequiredParameterValues(void) RecoveryRequiresIntParameter("max_connections", MaxConnections, ControlFile->MaxConnections); - RecoveryRequiresIntParameter("max_prepared_xacts", + RecoveryRequiresIntParameter("max_prepared_transactions", max_prepared_xacts, ControlFile->max_prepared_xacts); - RecoveryRequiresIntParameter("max_locks_per_xact", + RecoveryRequiresIntParameter("max_locks_per_transaction", max_locks_per_xact, ControlFile->max_locks_per_xact); } -- 2.40.0