From: | Dilip Kumar <dilipbalaut(at)gmail(dot)com> |
---|---|
To: | Ashutosh Sharma <ashu(dot)coek88(at)gmail(dot)com> |
Cc: | Neha Sharma <neha(dot)sharma(at)enterprisedb(dot)com>, Greg Nancarrow <gregn4422(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: [Proposal] Fully WAL logged CREATE DATABASE - No Checkpoints |
Date: | 2021-12-12 08:09:37 |
Message-ID: | CAFiTN-szcEFKktcSALkjZjoEbNqiJCB75uojX1mv0TPuFObK-w@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Dec 10, 2021 at 7:39 AM Ashutosh Sharma <ashu(dot)coek88(at)gmail(dot)com> wrote:
>>
>> Logfile Snippet:
>> 2021-12-09 17:49:18.110 +04 [18151] PANIC: could not fsync file "base/116398/116400": No such file or directory
>> 2021-12-09 17:49:19.105 +04 [18150] LOG: checkpointer process (PID 18151) was terminated by signal 6: Aborted
>> 2021-12-09 17:49:19.105 +04 [18150] LOG: terminating any other active server processes
>
>
> This is different from the issue you raised earlier. As Dilip said, we need to unregister sync requests for files that got successfully copied to the target database, but the overall alter database statement failed. We are doing this when the database is created successfully, but not when it fails.
> Probably doing the same inside the cleanup function movedb_failure_callback() should fix the problem.
Correct, I have done this cleanup, apart from this we have dropped the
fsyc request in create database failure case as well and also need to
drop buffer in error case of creatdb as well as movedb. I have also
fixed the other issue for which you gave the patch (a bit differently)
basically, in case of movedb the source and destination dboid are same
so we don't need an additional parameter and also readjusted the
conditions to avoid nested if.
--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com
Attachment | Content-Type | Size |
---|---|---|
v8-0001-Refactor-relmap-load-and-relmap-write-functions.patch | text/x-patch | 7.9 KB |
v8-0002-Extend-relmap-interfaces.patch | text/x-patch | 8.3 KB |
v8-0005-New-interface-to-lock-relation-id.patch | text/x-patch | 2.2 KB |
v8-0004-Extend-bufmgr-interfaces.patch | text/x-patch | 7.2 KB |
v8-0003-Refactor-index_copy_data.patch | text/x-patch | 5.3 KB |
v8-0007-WAL-logged-CREATE-DATABASE.patch | text/x-patch | 29.5 KB |
v8-0006-Extend-ForgetDatabaseSyncRequests-interface.patch | text/x-patch | 2.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bharath Rupireddy | 2021-12-12 12:38:05 | Re: add recovery, backup, archive, streaming etc. activity messages to server logs along with ps display |
Previous Message | Noah Misch | 2021-12-12 07:34:11 | Re: Probable memory leak with ECPG and AIX |