From: | vignesh C <vignesh21(at)gmail(dot)com> |
---|---|
To: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
Cc: | Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andres Freund <andres(at)anarazel(dot)de>, Craig Ringer <craig(dot)ringer(at)enterprisedb(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Printing backtrace of postgres processes |
Date: | 2021-11-19 10:37:05 |
Message-ID: | CALDaNm0OZZayAdj8izW3Wz222ky=jWdTstuPD_yE_dk-YMC-hw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Nov 18, 2021 at 9:52 PM Justin Pryzby <pryzby(at)telsasoft(dot)com> wrote:
>
> On Wed, Nov 17, 2021 at 08:12:44PM +0530, vignesh C wrote:
> > Attached v14 patch has the fixes for the same.
>
> Thanks for updating the patch.
>
> I cleaned up the docs and comments. I think this could be nearly "Ready".
>
> If you like the changes in my "fixup" patch (0002 and 0004), you should be able
> to apply my 0002 on top of your 0001. I'm sure it'll cause some conflicts with
> your 2nd patch, though...
I have slightly modified and taken the changes. I have not taken a few
of the changes to keep it similar to pg_log_backend_memory_contexts.
> This doesn't bump the catversion, since that would cause the patch to fail in
> cfbot every time another commit updates catversion.
I have removed it, since it there in commit message it is ok.
> Your 0001 patch allows printing backtraces of autovacuum, but the doc says it's
> only for "backends". Should the change to autovacuum.c be moved to the 2nd
> patch ? Or, why is autovacuum so special that it should be handled in the
> first patch ?
I had separated the patches so that it is easier for review, I have
merged the changes as few rounds of review is done for the patch. Now
since the patch is merged, this change is handled.
The Attached v15 patch has the fixes for the same.
Regards,
Vignesh
Attachment | Content-Type | Size |
---|---|---|
v15-0001-Add-function-to-log-the-backtrace-of-the-specifi.patch | text/x-patch | 25.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Andrey V. Lepikhov | 2021-11-19 11:10:59 | Re: Global snapshots |
Previous Message | Amit Kapila | 2021-11-19 10:23:08 | Re: Non-superuser subscription owners |