On Wed, Apr 22, 2015 at 6:33 PM, Abhijit Menon-Sen <a...@2ndquadrant.com> wrote: > > At 2015-04-18 12:28:36 +0530, amit.kapil...@gmail.com wrote: > > > > I think you have missed to address the below point: > > > > >> 4) prefetch > > Updated patch attached, as well as the diff against the earlier version > just to make it easier to see the exact change I made (which is to copy > the skip logic from lazy_scan_heap, as you suggested). >
Few minor issues: 1. warning on windows \pgstatbloat.c(193): warning C4715: 'pgstatbloat' : not all control paths return a value 2. Failed to install pgstattuple--1.3.sql You have to name sql file as pgstattuple--1.3.sql rather than pgstattuple--1.2.sql Other than above 2 points, patch looks good to me. With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com
