From: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
---|---|
To: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
Cc: | pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Report runtimes in pg_upgrade verbose mode |
Date: | 2024-06-19 15:09:02 |
Message-ID: | ZnL0jvUvl0SDtMCD@nathan |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Jun 19, 2024 at 04:50:59PM +0200, Daniel Gustafsson wrote:
> When doing performance hacking on pg_upgrade it's often important to see
> individual runtimes to isolate changes. I've written versions of the attached
> patch numerous times, and I wouldn't be surprised if others have done the same.
Indeed: https://postgr.es/m/flat/20230727235134.GA3658499%40nathanxps13
> Is there any interest in adding something like the attached to pg_upgrade? The
> patch needs some cleaning and tidying up but I wanted to to gauge interest
> before investing time. I've added it to verbose mode mainly since it's not
> really all that informative for regular users I think.
I've been using 'ts -i' as Peter suggested [0], and that has worked
decently well. One other thing that I've noticed is that some potentially
long-running tasks don't have corresponding reports. For example, the
initial get_db_rel_and_slot_infos() on the old cluster doesn't report
anything, but that is often one of the most time-consuming steps.
[0] https://postgr.es/m/32d24bcf-9ac4-b10e-4aa2-da6975312eb2%40eisentraut.org
--
nathan
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2024-06-19 15:11:37 | Re: suspicious valgrind reports about radixtree/tidstore on arm64 |
Previous Message | vignesh C | 2024-06-19 15:03:37 | Re: Logical Replication of sequences |