On Thu, Mar 7, 2019 at 2:54 AM Robert Haas <robertmh...@gmail.com> wrote: > > On Wed, Mar 6, 2019 at 1:26 AM Masahiko Sawada <sawada.m...@gmail.com> wrote: > > Okay, attached the latest version of patch set. I've incorporated all > > comments I got and separated the patch for making vacuum options a > > Node (0001 patch). And the patch doesn't use parallel_workers. It > > might be proposed in the another form again in the future if > > requested. > > Why make it a Node? I mean I think a struct makes sense, but what's > the point of giving it a NodeTag? >
Well, the main point is consistency with other nodes and keep the code clean. Regards, -- Masahiko Sawada NIPPON TELEGRAPH AND TELEPHONE CORPORATION NTT Open Source Software Center