Fix a typo of comments in AutoVacLauncherMain

Started by Yugo Nagata12 months ago3 messageshackers
Jump to latest
#1Yugo Nagata
nagata@sraoss.co.jp

Hi,

While reading autovacuum.c, I found the following comments in
AutoVacLauncherMain() right before the call to rebuild_database_list();

* Create the initial database list. The invariant we want this list to
* keep is that it's ordered by decreasing next_time. As soon as an entry

However, I couldn't fnd any variable named "next_time". I believe it actually refers
to adl_next_worker and the comments should use "next_worker" as is done in other nereby
comments. I've attahed a patch to address this.

Regards,
Yugo Nagata

--
Yugo Nagata <nagata@sraoss.co.jp>

Attachments:

fix_comment_typo_autovacuum.patchtext/x-diff; name=fix_comment_typo_autovacuum.patchDownload+1-1
#2Masahiko Sawada
sawada.mshk@gmail.com
In reply to: Yugo Nagata (#1)
Re: Fix a typo of comments in AutoVacLauncherMain

On Fri, Aug 1, 2025 at 8:20 AM Yugo Nagata <nagata@sraoss.co.jp> wrote:

Hi,

While reading autovacuum.c, I found the following comments in
AutoVacLauncherMain() right before the call to rebuild_database_list();

* Create the initial database list. The invariant we want this list to
* keep is that it's ordered by decreasing next_time. As soon as an entry

However, I couldn't fnd any variable named "next_time". I believe it actually refers
to adl_next_worker and the comments should use "next_worker" as is done in other nereby
comments. I've attahed a patch to address this.

I agree with your analysis. WIll push the patch shortly, barring any objections.

Regards,

--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com

#3Yugo Nagata
nagata@sraoss.co.jp
In reply to: Masahiko Sawada (#2)
Re: Fix a typo of comments in AutoVacLauncherMain

On Fri, 1 Aug 2025 10:00:48 -0700
Masahiko Sawada <sawada.mshk@gmail.com> wrote:

On Fri, Aug 1, 2025 at 8:20 AM Yugo Nagata <nagata@sraoss.co.jp> wrote:

Hi,

While reading autovacuum.c, I found the following comments in
AutoVacLauncherMain() right before the call to rebuild_database_list();

* Create the initial database list. The invariant we want this list to
* keep is that it's ordered by decreasing next_time. As soon as an entry

However, I couldn't fnd any variable named "next_time". I believe it actually refers
to adl_next_worker and the comments should use "next_worker" as is done in other nereby
comments. I've attahed a patch to address this.

I agree with your analysis. WIll push the patch shortly, barring any objections.

Thank you!

Yugo Nagata
Regards,

--
Yugo Nagata <nagata@sraoss.co.jp>