postgres.git / summary / log / commit / refs

commit    308d4ec4c677a2ced29b82dde811b1cbba8c3201
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Thu Sep 06 01:49:11 2012 +0000
First released in: 9.3.0, 9.0.10, 9.1.6, 9.2.0

    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 52bc469f296..83d8f53a9a7 100644
--- a/src/backend/access/transam/xlog.c
+++ b/src/backend/access/transam/xlog.c
@@ -5990,9 +5990,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, \
@@ -6033,10 +6034,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);
 	}


[parent: 8a66a37d6df5]