On Wed, Mar 22, 2017 at 1:53 PM, Seki, Eiji <seki.e...@jp.fujitsu.com> wrote:
> > Thank you for your review, again. > > I think your proposals are better, so I reflected them. Thanks for the updated patch. Patch looks good to me. I marked it as "ready for committer". While reviewing this patch, I found that PGXACT->vacuumFlags variable name needs a rename because with the addition of PROC_IN_LOGICAL_DECODING flag "vacuumFlags" doesn't only use it for vacuum operation. I feel this variable can be renamed as just "flags", but anyway that is a different patch. Regards, Hari Babu Fujitsu Australia