Tighten short-circuit tests for deciding whether we need to invoke

tglsfdc · tglsfdc · commit 0966516b75f7 · 2004-01-16T20:51:30.000Z
tuptoaster.c --- fields that are compressed in-line are not a reason
to invoke the toaster.  Along the way, add a couple more htup.h macros
to eliminate confusing negated tests, and get rid of the already
vestigial TUPLE_TOASTER_ACTIVE symbol.
diff --git a/src/backend/access/common/heaptuple.c b/src/backend/access/common/heaptuple.c
@@ -9,7 +9,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/access/common/heaptuple.c,v 1.88 2003/11/29 19:51:39 pgsql Exp $
+ *	  $PostgreSQL: pgsql/src/backend/access/common/heaptuple.c,v 1.89 2004/01/16 20:51:30 tgl Exp $
  *
  * NOTES
  *	  The old interface functions have been converted to macros
@@ -303,7 +303,7 @@ nocachegetattr(HeapTuple tuple,
 			return fetchatt(att[attnum],
 							tp + att[attnum]->attcacheoff);
 		}
-		else if (!HeapTupleAllFixed(tuple))
+		else if (HeapTupleHasVarWidth(tuple))
 		{
 			int			j;
 
@@ -378,13 +378,10 @@ nocachegetattr(HeapTuple tuple,
 
 		for (i = 0; i < attnum; i++)
 		{
-			if (!HeapTupleNoNulls(tuple))
+			if (HeapTupleHasNulls(tuple) && att_isnull(i, bp))
 			{
-				if (att_isnull(i, bp))
-				{
-					usecache = false;
-					continue;
-				}
+				usecache = false;
+				continue;
 			}
 
 			/* If we know the next offset, we can skip the rest */
diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/access/heap/heapam.c,v 1.161 2004/01/07 18:56:24 neilc Exp $
+ *	  $PostgreSQL: pgsql/src/backend/access/heap/heapam.c,v 1.162 2004/01/16 20:51:30 tgl Exp $
  *
  *
  * INTERFACE ROUTINES
@@ -1091,16 +1091,13 @@ heap_insert(Relation relation, HeapTuple tup, CommandId cid)
 	HeapTupleHeaderSetCmin(tup->t_data, cid);
 	tup->t_tableOid = relation->rd_id;
 
-#ifdef TUPLE_TOASTER_ACTIVE
-
 	/*
 	 * If the new tuple is too big for storage or contains already toasted
-	 * attributes from some other relation, invoke the toaster.
+	 * out-of-line attributes from some other relation, invoke the toaster.
 	 */
-	if (HeapTupleHasExtended(tup) ||
+	if (HeapTupleHasExternal(tup) ||
 		(MAXALIGN(tup->t_len) > TOAST_TUPLE_THRESHOLD))
 		heap_tuple_toast_attrs(relation, tup, NULL);
-#endif
 
 	/* Find buffer to insert this tuple into */
 	buffer = RelationGetBufferForTuple(relation, tup->t_len, InvalidBuffer);
@@ -1352,17 +1349,14 @@ heap_delete(Relation relation, ItemPointer tid,
 
 	LockBuffer(buffer, BUFFER_LOCK_UNLOCK);
 
-#ifdef TUPLE_TOASTER_ACTIVE
-
 	/*
-	 * If the relation has toastable attributes, we need to delete no
-	 * longer needed items there too.  We have to do this before
-	 * WriteBuffer because we need to look at the contents of the tuple,
-	 * but it's OK to release the context lock on the buffer first.
+	 * If the tuple has toasted out-of-line attributes, we need to delete
+	 * those items too.  We have to do this before WriteBuffer because we need
+	 * to look at the contents of the tuple, but it's OK to release the
+	 * context lock on the buffer first.
 	 */
-	if (HeapTupleHasExtended(&tp))
-		heap_tuple_toast_attrs(relation, NULL, &(tp));
-#endif
+	if (HeapTupleHasExternal(&tp))
+		heap_tuple_toast_attrs(relation, NULL, &tp);
 
 	pgstat_count_heap_delete(&relation->pgstat_info);
 
@@ -1572,11 +1566,11 @@ heap_update(Relation relation, ItemPointer otid, HeapTuple newtup,
 	 * implement UNDO and will re-use transaction IDs after postmaster
 	 * startup.
 	 *
-	 * We need to invoke the toaster if there are already any toasted values
-	 * present, or if the new tuple is over-threshold.
+	 * We need to invoke the toaster if there are already any out-of-line
+	 * toasted values present, or if the new tuple is over-threshold.
 	 */
-	need_toast = (HeapTupleHasExtended(&oldtup) ||
-				  HeapTupleHasExtended(newtup) ||
+	need_toast = (HeapTupleHasExternal(&oldtup) ||
+				  HeapTupleHasExternal(newtup) ||
 				  (MAXALIGN(newtup->t_len) > TOAST_TUPLE_THRESHOLD));
 
 	newtupsize = MAXALIGN(newtup->t_len);
diff --git a/src/include/access/htup.h b/src/include/access/htup.h
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/access/htup.h,v 1.63 2003/11/29 22:40:55 pgsql Exp $
+ * $PostgreSQL: pgsql/src/include/access/htup.h,v 1.64 2004/01/16 20:51:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -436,9 +436,15 @@ typedef HeapTupleData *HeapTuple;
  */
 #define HeapTupleIsValid(tuple) PointerIsValid(tuple)
 
+#define HeapTupleHasNulls(tuple) \
+		(((tuple)->t_data->t_infomask & HEAP_HASNULL) != 0)
+
 #define HeapTupleNoNulls(tuple) \
 		(!((tuple)->t_data->t_infomask & HEAP_HASNULL))
 
+#define HeapTupleHasVarWidth(tuple) \
+		(((tuple)->t_data->t_infomask & HEAP_HASVARWIDTH) != 0)
+
 #define HeapTupleAllFixed(tuple) \
 		(!((tuple)->t_data->t_infomask & HEAP_HASVARWIDTH))
 
diff --git a/src/include/postgres.h b/src/include/postgres.h
@@ -10,7 +10,7 @@
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1995, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/postgres.h,v 1.67 2004/01/04 05:57:21 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/postgres.h,v 1.68 2004/01/16 20:51:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -59,8 +59,6 @@
  * TOASTed.
  * ----------------
  */
-#define TUPLE_TOASTER_ACTIVE
-
 typedef struct varattrib
 {
 	int32		va_header;		/* External/compressed storage */

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`*`
`10`	`10`	`*`
`11`	`11`	`* IDENTIFICATION`
`12`		`- * $PostgreSQL: pgsql/src/backend/access/common/heaptuple.c,v 1.88 2003/11/29 19:51:39 pgsql Exp $`
	`12`	`+ * $PostgreSQL: pgsql/src/backend/access/common/heaptuple.c,v 1.89 2004/01/16 20:51:30 tgl Exp $`
`13`	`13`	`*`
`14`	`14`	`* NOTES`
`15`	`15`	`* The old interface functions have been converted to macros`
`@@ -303,7 +303,7 @@ nocachegetattr(HeapTuple tuple,`
`303`	`303`	`return fetchatt(att[attnum],`
`304`	`304`	`tp + att[attnum]->attcacheoff);`
`305`	`305`	`}`
`306`		`- else if (!HeapTupleAllFixed(tuple))`
	`306`	`+ else if (HeapTupleHasVarWidth(tuple))`
`307`	`307`	`{`
`308`	`308`	`int j;`
`309`	`309`
`@@ -378,13 +378,10 @@ nocachegetattr(HeapTuple tuple,`
`378`	`378`
`379`	`379`	`for (i = 0; i < attnum; i++)`
`380`	`380`	`{`
`381`		`- if (!HeapTupleNoNulls(tuple))`
	`381`	`+ if (HeapTupleHasNulls(tuple) && att_isnull(i, bp))`
`382`	`382`	`{`
`383`		`- if (att_isnull(i, bp))`
`384`		`- {`
`385`		`- usecache = false;`
`386`		`- continue;`
`387`		`- }`
	`383`	`+ usecache = false;`
	`384`	`+ continue;`
`388`	`385`	`}`
`389`	`386`
`390`	`387`	`/* If we know the next offset, we can skip the rest */`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`	`* Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group`
`11`	`11`	`* Portions Copyright (c) 1995, Regents of the University of California`
`12`	`12`	`*`
`13`		`- * $PostgreSQL: pgsql/src/include/postgres.h,v 1.67 2004/01/04 05:57:21 tgl Exp $`
	`13`	`+ * $PostgreSQL: pgsql/src/include/postgres.h,v 1.68 2004/01/16 20:51:30 tgl Exp $`
`14`	`14`	`*`
`15`	`15`	`*-------------------------------------------------------------------------`
`16`	`16`	`*/`
`@@ -59,8 +59,6 @@`
`59`	`59`	`* TOASTed.`
`60`	`60`	`* ----------------`
`61`	`61`	`*/`
`62`		`-#define TUPLE_TOASTER_ACTIVE`
`63`		`-`
`64`	`62`	`typedef struct varattrib`
`65`	`63`	`{`
`66`	`64`	`int32 va_header; /* External/compressed storage */`