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

Reply via email to