Reimplement nodeMaterial to use a temporary BufFile (or even memory, if the

materialized tupleset is small enough) instead of a temporary relation.
This was something I was thinking of doing anyway for performance, and Jan
says he needs it for TOAST because he doesn't want to cope with toasting
noname relations.  With this change, the 'noname table' support in heap.c
is dead code, and I have accordingly removed it.  Also clean up 'noname'
plan handling in planner --- nonames are either sort or materialize plans,
and it seems less confusing to handle them separately under those names.

1 files changed, 1 insertions, 11 deletions

diff --git a/src/backend/access/transam/xact.c b/src/backend/access/transam/xact.c
index cd20e231ecb..d6551cc9c9c 100644
--- a/src/backend/access/transam/xact.c
+++ b/src/backend/access/transam/xact.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/access/transam/xact.c,v 1.66 2000/06/08 22:36:54 momjian Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/access/transam/xact.c,v 1.67 2000/06/18 22:43:51 tgl Exp $
  *
  * NOTES
  *		Transaction aborts can now occur two ways:
@@ -878,14 +878,6 @@ StartTransaction()
 	AtStart_Locks();
 	AtStart_Memory();
 
-	/* --------------
-	   initialize temporary relations list
-	   the tempRelList is a list of temporary relations that
-	   are created in the course of the transactions
-	   they need to be destroyed properly at the end of the transactions
-	 */
-	InitNoNameRelList();
-
 	/* ----------------
 	 *	Tell the trigger manager to we're starting a transaction
 	 * ----------------
@@ -960,7 +952,6 @@ CommitTransaction()
 	AtCommit_Notify();
 
 	CloseSequences();
-	DropNoNameRels();
 	AtEOXact_portals();
 	RecordTransactionCommit();
 
@@ -1056,7 +1047,6 @@ AbortTransaction()
 		CommonSpecialPortalClose();
 	RecordTransactionAbort();
 	RelationPurgeLocalRelation(false);
-	DropNoNameRels();
 	invalidate_temp_relations();
 	AtEOXact_nbtree();
 	AtAbort_Cache();