On Wed, Jul 15, 2020 at 6:21 PM Dilip Kumar <dilipbal...@gmail.com> wrote: > > On Wed, Jul 15, 2020 at 6:17 PM Bharath Rupireddy > <bharath.rupireddyforpostg...@gmail.com> wrote: > > > > Hi, > > > > In ApplyLauncherMain, it seems like we are having SIGTERM signal > > mapped for config reload. I think we should be having SIGHUP for > > SignalHandlerForConfigReload(). Otherwise we miss to take the updated > > value for wal_retrieve_retry_interval in ApplyLauncherMain. > > > > Attached is a patch having this change. > > > > Thoughts? > > Yeah, it just looks like a typo so your fix looks good to me. >
+1. I will commit this tomorrow unless someone thinks otherwise. -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com
