Dear Amit,
Thank you for updating the patch. I have also checked the patch
and basically it has worked well. Almost all things I found were modified
by v4.
One comment: while setting logical_decoding_mode to wrong value,
I got unfriendly ERROR message.
```
postgres=# SET logical_decoding_mode = 1;
ERROR: invalid value for parameter "logical_decoding_mode": "1"
HINT: Available values: , immediate
```
Here all acceptable enum should be output as HINT, but we could not see the empty string.
Should we modify config_enum_get_options() for treating empty string, maybe
like (empty)? Or we can ignore now.
Best Regards,
Hayato Kuroda
FUJITSU LIMITED