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;