On Mon, Nov 15, 2021 at 10:04 PM vignesh C <vignesh21@gmail.com> wrote:
>
> On Mon, Nov 15, 2021 at 7:47 AM Bharath Rupireddy
> <bharath.rupireddyforpostgres@gmail.com> wrote:
> >
> > On Fri, Nov 5, 2021 at 11:12 AM Bharath Rupireddy
> > <bharath.rupireddyforpostgres@gmail.com> wrote:
> > > PSA v2 patch and review it.
> >
> > I've modified the docs part a bit, please consider v3 for review.
>
> Thanks for the update patch, Few comments:
> 1) Should we change "CHECK_FOR_INTERRUPTS()" to
> "CHECK_FOR_INTERRUPTS() or process specific interrupt handlers"
Done.
> 2) Should we mention Postmaster process also along with logger and
> statistics collector process
> + <glossterm linkend="glossary-backend">backend</glossterm> or the
> + <glossterm linkend="glossary-wal-sender">WAL sender</glossterm> or the
> + <glossterm linkend="glossary-auxiliary-proc">auxiliary
> process</glossterm>
> + with the specified process ID. All of the
> + <glossterm linkend="glossary-auxiliary-proc">auxiliary
> processes</glossterm>
> + are supported except the <glossterm
> linkend="glossary-logger">logger</glossterm>
> + and the <glossterm
> linkend="glossary-stats-collector">statistics collector</glossterm>
> + as they are not connected to shared memory the function can
> not make requests.
> + The backtrace will be logged at <literal>LOG</literal> message level.
> + They will appear in the server log based on the log configuration set
> + (See <xref linkend="runtime-config-logging"/> for more information),
> + but will not be sent to the client regardless of
Done.
Attaching v4 patch, please review it further.
Regards,
Bharath Rupireddy.
