On Wed, Feb 19, 2020 at 11:16 AM Peter Geoghegan <p...@bowt.ie> wrote: > On Wed, Feb 19, 2020 at 8:14 AM Anastasia Lubennikova > <a.lubennik...@postgrespro.ru> wrote: > > Thank you for this work. I've looked through the patches and they seem > > to be ready for commit. > > I haven't yet read recent documentation and readme changes, so maybe > > I'll send some more feedback tomorrow. > > Great.
I should add: I plan to commit the patch within the next 7 days. I believe that the design of deduplication itself is solid; it has many more strengths than weaknesses. It works in a way that complements the existing approach to page splits. The optimization can easily be turned off (and easily turned back on again). contrib/amcheck can detect almost any possible form of corruption that could affect a B-Tree index that has posting list tuples. I have spent months microbenchmarking every little aspect of this patch in isolation. I've also spent a lot of time on conventional benchmarking. It seems quite possible that somebody won't like some aspect of the user interface. I am more than willing to work with other contributors on any issue in that area that comes to light. I don't see any point in waiting for other hackers to speak up before the patch is committed, though. Anastasia posted the first version of this patch in August of 2015, and there have been over 30 revisions of it since the project was revived in 2019. Everyone has been given ample opportunity to offer input. -- Peter Geoghegan