Re: vacuumdb --analyze-only does not need to issue VACUUM (ONLY_DATABASE_STATS) ?

From: Mircea Cadariu <cadariu(dot)mircea(at)gmail(dot)com>
To: pgsql-hackers(at)lists(dot)postgresql(dot)org
Cc: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
Subject: Re: vacuumdb --analyze-only does not need to issue VACUUM (ONLY_DATABASE_STATS) ?
Date: 2025-09-16 13:05:45
Message-ID: 175802794504.1038.6545080940694370040.pgcf@coridan.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

The following review has been posted through the commitfest application:
make installcheck-world: tested, passed
Implements feature: tested, passed
Spec compliant: tested, passed
Documentation: tested, passed

Hi,

I tested the patch with log_statement = 'all' and I confirm I do not see the VACUUM log line anymore.

You could consider adding the following representative test in src/bin/scripts/t/100_vacuumdb.pl:
$node->issues_sql_unlike(
[ 'vacuumdb', '--analyze-only', 'postgres' ],
qr/statement: VACUUM.*;/,
'vacuumdb --analyze-only skips vacuum');

Kind regards,
Mircea Cadariu

The new status of this patch is: Waiting on Author

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message torikoshia 2025-09-16 13:30:31 Re: RFC: Logging plan of the running query
Previous Message Matthias van de Meent 2025-09-16 12:56:09 Re: Incorrect result of bitmap heap scan.