[pgpool-hackers: 4135] Re: Proposal: enable parameters related to logging_collector of RPM packages by default

[Bo Peng]

I have pushed it to master branch.
https://git.postgresql.org/gitweb/?p=pgpool2.git;a=commit;h=697c707c69f2af6f6141bd8e53b21a9de893c541

On Thu, 13 Jan 2022 15:55:29 +0900 (JST)
Tatsuo Ishii <ishii at sraoss.co.jp> wrote:

> >> But RPM users need to do more. From the Pgpool-II doc, it seems
> >> following settings are needed to make it workable.
> >> 
> >> log_destination = 'stderr'
> >> logging_collector = on
> >> log_directory = '/var/log/pgpool_log'
> >> log_filename = 'pgpool-%Y-%m-%d_%H%M%S.log'
> >> log_truncate_on_rotation = on
> >> log_rotation_age = 1d
> >> log_rotation_size = 10MB
> >> 
> >> If so why do you enable only logging_collector?
> >> Don't you need to set other log_* parameter as well?
> > 
> > Yes.
> > We also need to set other parameters below:
> > 
> >   log_destination = 'stderr'
> >   logging_collector = on
> >   log_directory = '/var/log/pgpool_log'
> >   log_filename = 'postgresql-%a.log'
> >   log_rotation_age = 1d
> >   log_rotation_size = 0
> >   log_truncate_on_rotation = on
> 
> So are you going to set these parameters in the RPM package?  If so, I
> think you also need to create /var/log/pgpool_log directory in the RPM
> script. What do you think?
> 
> Best reagards,
> --
> Tatsuo Ishii
> SRA OSS, Inc. Japan
> English: http://www.sraoss.co.jp/index_en.php
> Japanese:http://www.sraoss.co.jp


-- 
Bo Peng <pengbo at sraoss.co.jp>
SRA OSS, Inc. Japan
http://www.sraoss.co.jp/