mirror of
https://github.com/postgres/postgres.git
synced 2025-08-18 12:22:09 +03:00
Change CREATE TABLE so that column default expressions coming from different
inheritance parent tables are compared using equal(), instead of doing strcmp() on the nodeToString representation. The old implementation was always a tad cheesy, and it finally fails completely as of 8.4, now that the node tree might contain syntax location information. equal() knows it's supposed to ignore those fields, but strcmp() hardly can. Per recent report from Scott Ribe.
This commit is contained in:
@@ -8,7 +8,7 @@
|
||||
*
|
||||
*
|
||||
* IDENTIFICATION
|
||||
* $PostgreSQL: pgsql/src/backend/commands/tablecmds.c,v 1.288.2.1 2009/08/07 15:28:07 tgl Exp $
|
||||
* $PostgreSQL: pgsql/src/backend/commands/tablecmds.c,v 1.288.2.2 2009/10/06 00:55:34 tgl Exp $
|
||||
*
|
||||
*-------------------------------------------------------------------------
|
||||
*/
|
||||
@@ -486,7 +486,7 @@ DefineRelation(CreateStmt *stmt, char relkind)
|
||||
cooked->contype = CONSTR_DEFAULT;
|
||||
cooked->name = NULL;
|
||||
cooked->attnum = attnum;
|
||||
cooked->expr = stringToNode(colDef->cooked_default);
|
||||
cooked->expr = colDef->cooked_default;
|
||||
cooked->is_local = true; /* not used for defaults */
|
||||
cooked->inhcount = 0; /* ditto */
|
||||
cookedDefaults = lappend(cookedDefaults, cooked);
|
||||
@@ -1136,8 +1136,8 @@ MergeAttributes(List *schema, List *supers, bool istemp,
|
||||
List *constraints = NIL;
|
||||
int parentsWithOids = 0;
|
||||
bool have_bogus_defaults = false;
|
||||
char *bogus_marker = "Bogus!"; /* marks conflicting defaults */
|
||||
int child_attno;
|
||||
static Node bogus_marker = { 0 }; /* marks conflicting defaults */
|
||||
|
||||
/*
|
||||
* Check for and reject tables with too many columns. We perform this
|
||||
@@ -1321,7 +1321,7 @@ MergeAttributes(List *schema, List *supers, bool istemp,
|
||||
*/
|
||||
if (attribute->atthasdef)
|
||||
{
|
||||
char *this_default = NULL;
|
||||
Node *this_default = NULL;
|
||||
AttrDefault *attrdef;
|
||||
int i;
|
||||
|
||||
@@ -1332,7 +1332,7 @@ MergeAttributes(List *schema, List *supers, bool istemp,
|
||||
{
|
||||
if (attrdef[i].adnum == parent_attno)
|
||||
{
|
||||
this_default = attrdef[i].adbin;
|
||||
this_default = stringToNode(attrdef[i].adbin);
|
||||
break;
|
||||
}
|
||||
}
|
||||
@@ -1350,10 +1350,10 @@ MergeAttributes(List *schema, List *supers, bool istemp,
|
||||
*/
|
||||
Assert(def->raw_default == NULL);
|
||||
if (def->cooked_default == NULL)
|
||||
def->cooked_default = pstrdup(this_default);
|
||||
else if (strcmp(def->cooked_default, this_default) != 0)
|
||||
def->cooked_default = this_default;
|
||||
else if (!equal(def->cooked_default, this_default))
|
||||
{
|
||||
def->cooked_default = bogus_marker;
|
||||
def->cooked_default = &bogus_marker;
|
||||
have_bogus_defaults = true;
|
||||
}
|
||||
}
|
||||
@@ -1492,7 +1492,7 @@ MergeAttributes(List *schema, List *supers, bool istemp,
|
||||
{
|
||||
ColumnDef *def = lfirst(entry);
|
||||
|
||||
if (def->cooked_default == bogus_marker)
|
||||
if (def->cooked_default == &bogus_marker)
|
||||
ereport(ERROR,
|
||||
(errcode(ERRCODE_INVALID_COLUMN_DEFINITION),
|
||||
errmsg("column \"%s\" inherits conflicting default values",
|
||||
|
@@ -15,7 +15,7 @@
|
||||
* Portions Copyright (c) 1994, Regents of the University of California
|
||||
*
|
||||
* IDENTIFICATION
|
||||
* $PostgreSQL: pgsql/src/backend/nodes/copyfuncs.c,v 1.432 2009/06/18 01:27:02 tgl Exp $
|
||||
* $PostgreSQL: pgsql/src/backend/nodes/copyfuncs.c,v 1.432.2.1 2009/10/06 00:55:34 tgl Exp $
|
||||
*
|
||||
*-------------------------------------------------------------------------
|
||||
*/
|
||||
@@ -2073,7 +2073,7 @@ _copyColumnDef(ColumnDef *from)
|
||||
COPY_SCALAR_FIELD(is_local);
|
||||
COPY_SCALAR_FIELD(is_not_null);
|
||||
COPY_NODE_FIELD(raw_default);
|
||||
COPY_STRING_FIELD(cooked_default);
|
||||
COPY_NODE_FIELD(cooked_default);
|
||||
COPY_NODE_FIELD(constraints);
|
||||
|
||||
return newnode;
|
||||
|
@@ -22,7 +22,7 @@
|
||||
* Portions Copyright (c) 1994, Regents of the University of California
|
||||
*
|
||||
* IDENTIFICATION
|
||||
* $PostgreSQL: pgsql/src/backend/nodes/equalfuncs.c,v 1.355 2009/06/18 01:27:02 tgl Exp $
|
||||
* $PostgreSQL: pgsql/src/backend/nodes/equalfuncs.c,v 1.355.2.1 2009/10/06 00:55:34 tgl Exp $
|
||||
*
|
||||
*-------------------------------------------------------------------------
|
||||
*/
|
||||
@@ -2052,7 +2052,7 @@ _equalColumnDef(ColumnDef *a, ColumnDef *b)
|
||||
COMPARE_SCALAR_FIELD(is_local);
|
||||
COMPARE_SCALAR_FIELD(is_not_null);
|
||||
COMPARE_NODE_FIELD(raw_default);
|
||||
COMPARE_STRING_FIELD(cooked_default);
|
||||
COMPARE_NODE_FIELD(cooked_default);
|
||||
COMPARE_NODE_FIELD(constraints);
|
||||
|
||||
return true;
|
||||
|
@@ -8,7 +8,7 @@
|
||||
*
|
||||
*
|
||||
* IDENTIFICATION
|
||||
* $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.360 2009/06/11 14:48:58 momjian Exp $
|
||||
* $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.360.2.1 2009/10/06 00:55:34 tgl Exp $
|
||||
*
|
||||
* NOTES
|
||||
* Every node type that can appear in stored rules' parsetrees *must*
|
||||
@@ -1837,7 +1837,7 @@ _outColumnDef(StringInfo str, ColumnDef *node)
|
||||
WRITE_BOOL_FIELD(is_local);
|
||||
WRITE_BOOL_FIELD(is_not_null);
|
||||
WRITE_NODE_FIELD(raw_default);
|
||||
WRITE_STRING_FIELD(cooked_default);
|
||||
WRITE_NODE_FIELD(cooked_default);
|
||||
WRITE_NODE_FIELD(constraints);
|
||||
}
|
||||
|
||||
|
@@ -19,7 +19,7 @@
|
||||
* Portions Copyright (c) 1996-2009, PostgreSQL Global Development Group
|
||||
* Portions Copyright (c) 1994, Regents of the University of California
|
||||
*
|
||||
* $PostgreSQL: pgsql/src/backend/parser/parse_utilcmd.c,v 2.21 2009/06/11 14:49:00 momjian Exp $
|
||||
* $PostgreSQL: pgsql/src/backend/parser/parse_utilcmd.c,v 2.21.2.1 2009/10/06 00:55:35 tgl Exp $
|
||||
*
|
||||
*-------------------------------------------------------------------------
|
||||
*/
|
||||
@@ -642,7 +642,7 @@ transformInhRelation(ParseState *pstate, CreateStmtContext *cxt,
|
||||
*/
|
||||
if (attribute->atthasdef && including_defaults)
|
||||
{
|
||||
char *this_default = NULL;
|
||||
Node *this_default = NULL;
|
||||
AttrDefault *attrdef;
|
||||
int i;
|
||||
|
||||
@@ -653,7 +653,7 @@ transformInhRelation(ParseState *pstate, CreateStmtContext *cxt,
|
||||
{
|
||||
if (attrdef[i].adnum == parent_attno)
|
||||
{
|
||||
this_default = attrdef[i].adbin;
|
||||
this_default = stringToNode(attrdef[i].adbin);
|
||||
break;
|
||||
}
|
||||
}
|
||||
@@ -664,7 +664,7 @@ transformInhRelation(ParseState *pstate, CreateStmtContext *cxt,
|
||||
* but it can't; so default is ready to apply to child.
|
||||
*/
|
||||
|
||||
def->cooked_default = pstrdup(this_default);
|
||||
def->cooked_default = this_default;
|
||||
}
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user