Re: [HACKERS] heap_markpos and heap_restrpos

2006-09-26 Thread Tom Lane
Heikki Linnakangas <[EMAIL PROTECTED]> writes: > What's the purpose of mark/restrpos in heapam.c? It's deadwood --- see the comment for ExecSupportsMarkRestore: /* * ExecSupportsMarkRestore - does a plan type support mark/restore? * * XXX Ideally, all plan node types would support mark/restore

[HACKERS] heap_markpos and heap_restrpos

2006-09-26 Thread Heikki Linnakangas
What's the purpose of mark/restrpos in heapam.c? AFAIK, mark/restore is only used by merge joins, and you can't feed a merge join from a heap scan because merge join requires sorted input. Unless I'm missing something, heap_markpos and heap_restrpos are dead code. -- Heikki Linnakangas Ente