Fix a typo of mod_since_analyze in do_analyze_rel
Started by Yugo Nagata5 months ago2 messages
Hi,
I found comments in do_analyze_rel that refers to "change_since_analyze".
I believe that is a typo and shoud be "mod_since_analyze".
I've attached a patch to fix it.
Regards,
Yugo Nagata
--
Yugo Nagata <nagata@sraoss.co.jp>
Attachments:
fix_typo_mod_since_analyze.patchtext/x-diff; name=fix_typo_mod_since_analyze.patchDownload
diff --git a/src/backend/commands/analyze.c b/src/backend/commands/analyze.c
index 7111d5d5334..29daa10742e 100644
--- a/src/backend/commands/analyze.c
+++ b/src/backend/commands/analyze.c
@@ -690,7 +690,7 @@ do_analyze_rel(Relation onerel, const VacuumParams params,
* only do it for inherited stats. (We're never called for not-inherited
* stats on partitioned tables anyway.)
*
- * Reset the changes_since_analyze counter only if we analyzed all
+ * Reset the mod_since_analyze counter only if we analyzed all
* columns; otherwise, there is still work for auto-analyze to do.
*/
if (!inh)
Re: Fix a typo of mod_since_analyze in do_analyze_rel
On Mon, 4 Aug 2025 at 17:01, Yugo Nagata <nagata@sraoss.co.jp> wrote:
I found comments in do_analyze_rel that refers to "change_since_analyze".
I believe that is a typo and shoud be "mod_since_analyze".
I've attached a patch to fix it.
Yeah, that looks unintentional. I've pushed your fix.
Thanks
David