[pgpool-hackers: 2969] Re: changing fail-over to failover
m.usama at gmail.com
Sun Aug 19 18:51:54 JST 2018
On Sat, Aug 18, 2018 at 5:07 AM Tatsuo Ishii <ishii at sraoss.co.jp> wrote:
> Hi Usama,
> > Hi Ishii-San
> > Please find the attached patch to fix mistakes where "fail_over" and
> > "fail-over" were used instead of the "failover". And that also includes a
> > wrongly spelled configuration parameter "fail_over_on_backend_error",
> > The patch throws the warning message when old config name
> > fail_over_on_backend_error is used instead of failover_on_backend_error
> > suggests to use the correct name, while using the old config variable
> > has no effect.
> > What do you think of the approach to throw a waring message and making
> > old variable name ineffective.
> Looks good to me.
Thanks for the review, I have pushed it to master branch.
> Best regards,
> Tatsuo Ishii
> SRA OSS, Inc. Japan
> English: http://www.sraoss.co.jp/index_en.php
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the pgpool-hackers