On April 22, 2015 06:04:42 PM Payal Singh wrote: > The following review has been posted through the commitfest application: > make installcheck-world: not tested > Implements feature: tested, failed > Spec compliant: not tested > Documentation: tested, failed > > Error in postgresql.conf gives the expected result on pg_ctl reload, > although errors in pg_hba.conf file don't. Like before, reload completes > fine without any information that pg_hba failed to load and only > information is present in the log file. I'm assuming pg_ctl reload should > prompt user if file fails to load irrespective of which file it is - > postgresql.conf or pg_hba.conf.
Will fix. Not hard, just move the code that writes the .pid file to after the pg_hba reload. > > There is no documentation change so far, but I guess that's not yet > necessary. I will update the page for pg_ctl. At least the behaviour of -w/-W in relation to the reload command needs explained. > > gmake check passed all tests. > > The new status of this patch is: Waiting on Author jan -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
