From 6ec0753146c5ca69a9f1b9eee229c8fa7a55b2b0 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Wed, 27 May 2009 20:42:29 +0000 Subject: Ignore RECHECK in CREATE OPERATOR CLASS, just throwing a NOTICE, instead of throwing an error as 8.4 had been doing. The error interfered with porting old database definitions (particularly for pg_migrator) without really buying any safety. Per bug #4817 and subsequent discussion. --- src/backend/parser/gram.y | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'src/backend/parser') diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y index 7e2609eecc1..544b69246ed 100644 --- a/src/backend/parser/gram.y +++ b/src/backend/parser/gram.y @@ -11,7 +11,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/parser/gram.y,v 2.663 2009/04/28 09:09:41 heikki Exp $ + * $PostgreSQL: pgsql/src/backend/parser/gram.y,v 2.664 2009/05/27 20:42:29 tgl Exp $ * * HISTORY * AUTHOR DATE MAJOR EVENT @@ -3639,9 +3639,14 @@ opt_opfamily: FAMILY any_name { $$ = $2; } opt_recheck: RECHECK { - ereport(ERROR, + /* + * RECHECK no longer does anything in opclass definitions, + * but we still accept it to ease porting of old database + * dumps. + */ + ereport(NOTICE, (errcode(ERRCODE_FEATURE_NOT_SUPPORTED), - errmsg("RECHECK is no longer supported"), + errmsg("RECHECK is no longer required"), errhint("Update your data type."), scanner_errposition(@1))); $$ = TRUE; -- cgit v1.2.3