Re: [HACKERS] code cleanup: ss_currentScanDesc

2010-02-25 Thread Tom Lane
Takahiro Itagaki writes: > ScanState.ss_currentScanDesc is currently used by only SeqScan and > BitmapHeapScan. Other scan nodes don't use the field at all, right? > Can we move the field into SeqScanState and BitmapHeapScanState > for code cleanup? This change will not improve any performance, >

[HACKERS] code cleanup: ss_currentScanDesc

2010-02-25 Thread Takahiro Itagaki
ScanState.ss_currentScanDesc is currently used by only SeqScan and BitmapHeapScan. Other scan nodes don't use the field at all, right? Can we move the field into SeqScanState and BitmapHeapScanState for code cleanup? This change will not improve any performance, but it can clear up what we do actu