Hi,
This appears to have worked, but required a patch to your patch -- which I've attached.
The new version passes regression tests in a manner very similar to the old,
so I'm assuming all is well, here, and that you mistakenly sent me one
of the features I'm not ready for yet... (CONSTR_NULL was undefined).
I'm just a novice here, so please, please, correct it if necessary!
Thanks very much for the quick service!
*** analyze.c 1998/12/29 18:43:50 1.2
--- analyze.c 1998/12/29 18:48:53
***************
*** 7,13 **** * * * IDENTIFICATION
! * $Header: /usr/local/ports/databases/postgres-6.4.1/work/postgresql-6.4.1/src/backend/parser/RCS/analyze.c,v
1.21998/12/29 18:43:50 pgsql Exp $ * *------------------------------------------------------------------------- */
--- 7,13 ---- * * * IDENTIFICATION
! * $Header: /usr/local/ports/databases/postgres-6.4.1/work/postgresql-6.4.1/src/backend/parser/RCS/analyze.c,v
1.31998/12/29 18:48:50 pgsql Exp $ * *------------------------------------------------------------------------- */
***************
*** 592,606 **** constraint = lfirst(clist); switch
(constraint->contype) {
- case CONSTR_NULL:
- /* We should mark this explicitly,
- * so we can tell if NULL and NOT NULL are both specified
- */
- if (column->is_not_null)
- elog(ERROR, "CREATE TABLE/(NOT) NULL conflicting declaration"
- " for %s.%s", stmt->relname, column->colname);
- column->is_not_null = FALSE;
- break; case CONSTR_NOTNULL:
if (column->is_not_null)
--- 592,597 ----