Horton

Thread

Commits

  1. Remove redundant HandleWalWriterInterrupts().

  2. Remove redundant call to pgstat_report_wal()

  1. Remove redundant HandleWalWriterInterrupts()

    Fujii Masao <masao.fujii@gmail.com> — 2024-01-24T12:20:17Z

    Hi,

Because of commit 1bdd54e662, the code of HandleWalWriterInterrupts()
became the same as HandleMainLoopInterrupts(). So I'd like to propose to
remove HandleWalWriterInterrupts() and make walwriter use
HandleMainLoopInterrupts() instead for improved code simplicity. Thought?

Patch attached.

Regards,

-- 
Fujii Masao

  2. Re: Remove redundant HandleWalWriterInterrupts()

    Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> — 2024-01-24T14:00:17Z

    On Wed, Jan 24, 2024 at 5:50 PM Fujii Masao <masao.fujii@gmail.com> wrote:
>
> Hi,
>
> Because of commit 1bdd54e662, the code of HandleWalWriterInterrupts()
> became the same as HandleMainLoopInterrupts(). So I'd like to propose to
> remove HandleWalWriterInterrupts() and make walwriter use
> HandleMainLoopInterrupts() instead for improved code simplicity. Thought?
>
> Patch attached.

Nice catch. Indeed they both are the same after commit 1bdd54e662. The
patch LGTM.

-- 
Bharath Rupireddy
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com

  3. Re: Remove redundant HandleWalWriterInterrupts()

    Nathan Bossart <nathandbossart@gmail.com> — 2024-01-24T19:40:19Z

    On Wed, Jan 24, 2024 at 07:30:17PM +0530, Bharath Rupireddy wrote:
> On Wed, Jan 24, 2024 at 5:50 PM Fujii Masao <masao.fujii@gmail.com> wrote:
>> Because of commit 1bdd54e662, the code of HandleWalWriterInterrupts()
>> became the same as HandleMainLoopInterrupts(). So I'd like to propose to
>> remove HandleWalWriterInterrupts() and make walwriter use
>> HandleMainLoopInterrupts() instead for improved code simplicity. Thought?
>>
>> Patch attached.
> 
> Nice catch. Indeed they both are the same after commit 1bdd54e662. The
> patch LGTM.

LGTM, too.

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com

  4. Re: Remove redundant HandleWalWriterInterrupts()

    Fujii Masao <masao.fujii@gmail.com> — 2024-01-25T03:57:06Z

    On Thu, Jan 25, 2024 at 4:40 AM Nathan Bossart <nathandbossart@gmail.com> wrote:
>
> On Wed, Jan 24, 2024 at 07:30:17PM +0530, Bharath Rupireddy wrote:
> > On Wed, Jan 24, 2024 at 5:50 PM Fujii Masao <masao.fujii@gmail.com> wrote:
> >> Because of commit 1bdd54e662, the code of HandleWalWriterInterrupts()
> >> became the same as HandleMainLoopInterrupts(). So I'd like to propose to
> >> remove HandleWalWriterInterrupts() and make walwriter use
> >> HandleMainLoopInterrupts() instead for improved code simplicity. Thought?
> >>
> >> Patch attached.
> >
> > Nice catch. Indeed they both are the same after commit 1bdd54e662. The
> > patch LGTM.
>
> LGTM, too.

Thank you both for reviewing! I've pushed the patch.

Regards,

-- 
Fujii Masao