Remove incorrect 'Assert(targetList != NULL)'. An

INSERT ... DEFAULT VALUES statement does indeed have a null targetlist,
at least during parse and rewrite stages.

diff --git a/src/backend/rewrite/rewriteHandler.c b/src/backend/rewrite/rewriteHandler.c
index b13cb5e3ed1cd5db2f142db51cf15e8aa1681bb0..a76121aa823eec3402090fabca05c38fac061d8f 100644 (file)
--- a/src/backend/rewrite/rewriteHandler.c
+++ b/src/backend/rewrite/rewriteHandler.c
@@ -6,7 +6,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/rewrite/rewriteHandler.c,v 1.56 1999/09/18 19:07:19 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/rewrite/rewriteHandler.c,v 1.57 1999/09/19 17:20:58 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -2440,17 +2440,10 @@ RewriteQuery(Query *parsetree, bool *instead_flag, List **qual_products)
        if (event == CMD_UTILITY)
                return NIL;
 
-       /*
-        * only for a delete may the targetlist be NULL
-        */
-       if (event != CMD_DELETE)
-               Assert(parsetree->targetList != NULL);
-
-       result_relation = parsetree->resultRelation;
-
        /*
         * the statement is an update, insert or delete - fire rules on it.
         */
+       result_relation = parsetree->resultRelation;
        rt_entry = rt_fetch(result_relation, parsetree->rtable);
        rt_entry_relation = heap_openr(rt_entry->relname, AccessShareLock);
        rt_entry_locks = rt_entry_relation->rd_rules;