On Jan 9, 2019, at 4:47 PM, Michael Paquier <mich...@paquier.xyz> wrote: > That's much cleaner to me to clean up the field for safety before > starting the process. When requesting a WAL receiver to start, > slotname and conninfo gets zeroed before doing anything, you are right > that we could do without it actually.
Cool! I agree that cleaning up the field would make the logic cleaner. > That's a pattern used quite a lot for many GUCs, so that's quite > separate from this patch. Perhaps it would make sense to have a macro > for that purpose for GUCs, I am not sure if that's worth it though. Yeah. I think having such macro would make the code a bit cleaner. Tho the duplication of logic is not too significant.