From: | Greg Stark <stark(at)mit(dot)edu> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Noah Misch <noah(at)leadboat(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Temporary tables versus wraparound... again |
Date: | 2022-03-23 03:58:31 |
Message-ID: | CAM-w4HOFp0wbH6rXrt0Wr3L9s77WiPouVAh9N_U_zzhw8fJxOg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Here's a rebased patch. I split the test into a separate patch that I
would lean to dropping. But at least it applies now.
I did look into pg_stat_get_backend_pid() and I guess it would work
but going through the stats mechanism does seem like going the long
way around since we're already looking at the backendId info directly
here, we just weren't grabbing the pid.
I did make a small change, I renamed the checkTempNamespaceStatus()
function to checkTempNamespaceStatusAndPid(). It seems unlikely there
are any external consumers of this function (the only internal
consumer is autovacuum.c). But just in case I renamed it to protect
against any external modules failing from the added parameter.
Attachment | Content-Type | Size |
---|---|---|
v3-0002-Add-test-for-truncating-temp-tables-advancing-rel.patch | text/x-patch | 4.2 KB |
v3-0001-Update-relfrozenxmin-when-truncating-temp-tables.patch | text/x-patch | 12.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Zhihong Yu | 2022-03-23 04:04:03 | Re: freeing bms explicitly |
Previous Message | Amit Kapila | 2022-03-23 03:45:27 | Re: freeing bms explicitly |