On Wed, 25 Dec 2019 at 17:47, Masahiko Sawada <masahiko.saw...@2ndquadrant.com> wrote: > > On Tue, 24 Dec 2019 at 15:46, Masahiko Sawada > <masahiko.saw...@2ndquadrant.com> wrote: > > > > On Tue, 24 Dec 2019 at 15:44, Amit Kapila <amit.kapil...@gmail.com> wrote: > > > > > > On Tue, Dec 24, 2019 at 12:08 PM Masahiko Sawada > > > <masahiko.saw...@2ndquadrant.com> wrote: > > > > > > > > > > > > The first patches look good to me. I'm reviewing other patches and > > > > will post comments if there is. > > > > > > > > Oops I meant first "two" patches look good to me. > > > > > > > > Okay, feel free to address few comments raised by Mahendra along with > > > whatever you find. > > > > Thanks! > > > > I've attached updated patch set as the previous version patch set > conflicts to the current HEAD. This patch set incorporated the review > comments, a few fix and the patch for > PARALLEL_VACUUM_DISABLE_LEADER_PARTICIPATION. 0001 patch is the same > as previous version.
I verified my all review comments in v40 patch set. All are fixed. v40-0002-Add-a-parallel-option-to-the-VACUUM-command.patch doesn't apply on HEAD. Please send re-based patch. Thanks and Regards Mahendra Singh Thalor EnterpriseDB: http://www.enterprisedb.com