Re: Vacuum statistics

From: Alena Rybakina <a(dot)rybakina(at)postgrespro(dot)ru>
To: Kirill Reshke <reshkekirill(at)gmail(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Alexander Korotkov <aekorotkov(at)gmail(dot)com>, Jim Nasby <jnasby(at)upgrade(dot)com>, Ilia Evdokimov <ilya(dot)evdokimov(at)tantorlabs(dot)com>, Andrei Zubkov <zubkov(at)moonset(dot)ru>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Melanie Plageman <melanieplageman(at)gmail(dot)com>, jian he <jian(dot)universality(at)gmail(dot)com>, a(dot)lepikhov(at)postgrespro(dot)ru, Sami Imseih <samimseih(at)gmail(dot)com>
Subject: Re: Vacuum statistics
Date: 2025-03-12 19:36:39
Message-ID: 1dcd4262-188f-4734-9a26-1c2a5aaf3da7@postgrespro.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi!

On 10.03.2025 16:33, Kirill Reshke wrote:
> On Thu, 27 Feb 2025 at 23:00, Alena Rybakina<a(dot)rybakina(at)postgrespro(dot)ru> wrote:
>> Hi!
>> On 17.02.2025 17:46, Alena Rybakina wrote:
>>> On 04.02.2025 18:22, Alena Rybakina wrote:
>>>> Hi! Thank you for your review!
>>>>
>>>> On 02.02.2025 23:43, Alexander Korotkov wrote:
>>>>> On Mon, Jan 13, 2025 at 3:26 PM Alena Rybakina
>>>>> <a(dot)rybakina(at)postgrespro(dot)ru> wrote:
>>>>>> I noticed that the cfbot is bad, the reason seems to be related to
>>>>>> the lack of a parameter in
>>>>>> src/backend/utils/misc/postgresql.conf.sample. I added it, it
>>>>>> should help.
>>>>> The patch doesn't apply cleanly. Please rebase.
>>>> I rebased them.
>>> The patch needed a rebase again. There is nothing new since version
>>> 18, only a rebase.
>> The patch needed a new rebase.
>>
>> Sorry, but due to feeling unwell I picked up a patch from another thread
>> and squashed the patch for vacuum statistics for indexes and heaps in
>> the previous version. Now I fixed everything together with the rebase.
>>
>> --
>> Regards,
>> Alena Rybakina
>> Postgres Professional
> Hi!
> CI fails on this one[0]
>
> Is it a flap or a real problem caused by v20?
>
> ```
>
> SELECT relpages AS irp
> diff -U3 /tmp/cirrus-ci-build/src/test/regress/expected/vacuum_tables_and_db_statistics.out
> /tmp/cirrus-ci-build/build-32/testrun/recovery/027_stream_regress/data/results/vacuum_tables_and_db_statistics.out
> --- /tmp/cirrus-ci-build/src/test/regress/expected/vacuum_tables_and_db_statistics.out
> 2025-03-10 09:36:10.274799176 +0000
> +++ /tmp/cirrus-ci-build/build-32/testrun/recovery/027_stream_regress/data/results/vacuum_tables_and_db_statistics.out
> 2025-03-10 09:49:35.796596462 +0000
> @@ -65,7 +65,7 @@
> WHERE vt.relname = 'vestat' AND vt.relid = c.oid;
> relname | vm_new_frozen_pages | tuples_deleted | relpages |
> pages_scanned | pages_removed
> ---------+---------------------+----------------+----------+---------------+---------------
> - vestat | 0 | 0 | 455 |
> 0 | 0
> + vestat | 0 | 0 | 417 |
> 0 | 0
> (1 row)
>
> SELECT relpages AS rp
> === EOF ===
>
>
> ```
>
> [0]https://api.cirrus-ci.com/v1/artifact/task/5336493629112320/testrun/build-32/testrun/recovery/027_stream_regress/log/regress_log_027_stream_regress
Thank you for your help, I'll fix it soon.

--
Regards,
Alena Rybakina
Postgres Professional

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Sami Imseih 2025-03-12 19:38:06 Re: making EXPLAIN extensible
Previous Message Nathan Bossart 2025-03-12 19:34:06 Re: vacuumdb changes for stats import/export