Minor comment update.

The line changed is in direct contradiction with the comment (and
reality) of the code below this. The opposed comment starts at line
1548:

/*
 * Note: Normally one would think that we have to delete index tuples
 * associated with the heap tuple now..
 *
 * ... but in POSTGRES, we have no need to do this because the vacuum
daemon
 * automatically opens an index scan and deletes index tuples when it
 * finds deleted heap tuples. -cim 9/27/89
 */

Seems like its been wrong for a while...

Best Regards, Simon Riggs
Index: src/backend/executor/execMain.c
===================================================================
RCS file: /projects/cvsroot/pgsql/src/backend/executor/execMain.c,v
retrieving revision 1.257
diff -c -r1.257 execMain.c
*** src/backend/executor/execMain.c	14 Nov 2005 17:42:54 -0000	1.257
--- src/backend/executor/execMain.c	18 Nov 2005 00:10:21 -0000
***************
*** 1458,1465 ****
  /* ----------------------------------------------------------------
   *		ExecDelete
   *
!  *		DELETE is like UPDATE, we delete the tuple and its
!  *		index tuples.
   * ----------------------------------------------------------------
   */
  static void
--- 1458,1464 ----
  /* ----------------------------------------------------------------
   *		ExecDelete
   *
!  *		DELETE is like UPDATE, but we delete the tuple only; no indexes
   * ----------------------------------------------------------------
   */
  static void
---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster

Reply via email to