On Tue, Feb 7, 2017 at 11:53 AM, Beena Emerson <memissemer...@gmail.com> wrote: > Hello, > > Thank you for the updated patch. > > On Tue, Feb 7, 2017 at 10:44 AM, Mithun Cy <mithun...@enterprisedb.com> > wrote: >> >> Hi all, >> Here is the new patch which fixes all of above comments, I changed the >> design a bit now as below >> >> What is it? >> =========== >> A pair of bgwrokers one which automatically dumps buffer pool's block >> info at a given interval and another which loads those block into >> buffer pool when >> the server restarts. > > > Are 2 workers required? >
I think in the new design there is a provision of launching the worker dynamically to dump the buffers, so there seems to be a need of separate workers for loading and dumping the buffers. However, there is no explanation in the patch or otherwise when and why this needs a pair of workers. Also, if the dump interval is greater than zero, then do we really need to separately register a dynamic worker? -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers