Re: Tid scan improvements
Edmund Horner <ejrh00@gmail.com>
From: Edmund Horner <ejrh00@gmail.com>
To: David Rowley <david.rowley@2ndquadrant.com>
Cc: Andres Freund <andres@anarazel.de>, Thomas Munro <thomas.munro@gmail.com>, David Steele <david@pgmasters.net>, Tom Lane <tgl@sss.pgh.pa.us>, Tomas Vondra <tomas.vondra@2ndquadrant.com>, PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>
Date: 2019-08-01T05:33:57Z
Lists: pgsql-hackers
On Mon, 22 Jul 2019 at 19:44, Edmund Horner <ejrh00@gmail.com> wrote: > On Mon, 22 Jul 2019 at 19:25, David Rowley <david.rowley@2ndquadrant.com> > > On Sat, 20 Jul 2019 at 06:21, Andres Freund <andres@anarazel.de> wrote: > > > Yea, I was thinking of something like 2. We already have a few extra > > > types of scan nodes (bitmap heap and sample scan), it'd not be bad to > > > add another. And as you say, they can just share most of the guts: For > > > heap I'd just implement pagemode, and perhaps split heapgettup_pagemode > > > into two parts (one to do the page processing, the other to determine > > > the relevant page). > > > > > > You say that we'd need new fields in HeapScanDescData - not so sure > > > about that, it seems feasible to just provide the boundaries in the > > > call? But I think it'd also just be fine to have the additional fields. > > > > Thanks for explaining. > > > > I've set the CF entry for the patch back to waiting on author. > > > > I think if we get this part the way Andres would like it, then we're > > pretty close. > [...] > I'll have to spend a bit of time looking at heapgettup_pagemode to > figure how to split it as Andres suggests. Hi everyone, Sadly it is the end of the CF and I have not had much time to work on this. I'll probably get some time in the next month to look at the heapam changes. Should we move to CF? We have been in the CF cycle for almost a year now... Edmund
Commits
-
Add TID Range Scans to support efficient scanning ranges of TIDs
- bb437f995d47 14.0 landed
-
Improve planner's selectivity estimates for inequalities on CTID.
- f7111f72d2fd 12.0 landed