Here is the definition of relation_byte_size() in optimizer/path/costsize.c:

bmomjian · bmomjian · commit ed96bfde1829 · 2003-12-18T03:46:45.000Z
----------------------------------------------------------------------
/*
 * relation_byte_size
 *        Estimate the storage space in bytes for a given number of tuples
 *        of a given width (size in bytes).
 */
static double
relation_byte_size(double tuples, int width)
{
        return tuples * (MAXALIGN(width) + MAXALIGN(sizeof(HeapTupleData)));
}

----------------------------------------------------------------------

Shouldn't this be HeapTupleHeaderData and not HeapTupleData ?

(Of course, from a costing perspective these shouldn't be very different but ...)

Sailesh Krishnamurthy
diff --git a/src/backend/optimizer/path/costsize.c b/src/backend/optimizer/path/costsize.c
@@ -49,7 +49,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/optimizer/path/costsize.c,v 1.117 2003/12/03 17:45:07 tgl Exp $
+ *	  $PostgreSQL: pgsql/src/backend/optimizer/path/costsize.c,v 1.118 2003/12/18 03:46:45 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1969,7 +1969,7 @@ set_rel_width(Query *root, RelOptInfo *rel)
 static double
 relation_byte_size(double tuples, int width)
 {
-	return tuples * (MAXALIGN(width) + MAXALIGN(sizeof(HeapTupleData)));
+	return tuples * (MAXALIGN(width) + MAXALIGN(sizeof(HeapTupleHeaderData)));
 }
 
 /*

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@`
`49`	`49`	`* Portions Copyright (c) 1994, Regents of the University of California`
`50`	`50`	`*`
`51`	`51`	`* IDENTIFICATION`
`52`		`- * $PostgreSQL: pgsql/src/backend/optimizer/path/costsize.c,v 1.117 2003/12/03 17:45:07 tgl Exp $`
	`52`	`+ * $PostgreSQL: pgsql/src/backend/optimizer/path/costsize.c,v 1.118 2003/12/18 03:46:45 momjian Exp $`
`53`	`53`	`*`
`54`	`54`	`*-------------------------------------------------------------------------`
`55`	`55`	`*/`
`@@ -1969,7 +1969,7 @@ set_rel_width(Query root, RelOptInfo rel)`
`1969`	`1969`	`static double`
`1970`	`1970`	`relation_byte_size(double tuples, int width)`
`1971`	`1971`	`{`
`1972`		`- return tuples * (MAXALIGN(width) + MAXALIGN(sizeof(HeapTupleData)));`
	`1972`	`+ return tuples * (MAXALIGN(width) + MAXALIGN(sizeof(HeapTupleHeaderData)));`
`1973`	`1973`	`}`
`1974`	`1974`
`1975`	`1975`	`/*`