Re: [Proposal] Fully WAL logged CREATE DATABASE - No Checkpoints

On Wed, Dec 1, 2021 at 12:07 PM Greg Nancarrow <gregn4422(at)gmail(dot)com> wrote:
>
> On Thu, Nov 25, 2021 at 10:17 PM Dilip Kumar <dilipbalaut(at)gmail(dot)com> wrote:
> >
> > Thanks for the review and many valuable comments, I have fixed all of
> > them except this comment (/* If we got a cancel signal during the copy
> > of the data, quit */) because this looks fine to me. 0007, I have
> > dropped from the patchset for now. I have also included fixes for
> > comments given by John.
> >
>
> I found the following issue with the patches applied:
>
> A server crash occurs after the following sequence of commands:
>
> create tablespace tbsp1 location '<directory>/tbsp1';
> create tablespace tbsp2 location '<directory>/tbsp2';
> create database test1 tablespace tbsp1;
> create database test2 template test1 tablespace tbsp2;
> alter database test2 set tablespace tbsp1;
> checkpoint;
>
> The following type of message is seen in the server log:
>
> 2021-12-01 16:48:26.623 AEDT [67423] PANIC: could not fsync file
> "pg_tblspc/16385/PG_15_202111301/16387/3394": No such file or
> directory

Thanks a lot for testing this. From the error, it seems like some of
the old buffer w.r.t. the previous tablespace is not dropped after the
movedb. Actually, we are calling DropDatabaseBuffers() after copying
to a new tablespace and dropping all the buffers of this database
w.r.t the old tablespace. But seems something is missing, I will
reproduce this and try to fix it by tomorrow. I will also fix the
other review comments raised by you in the previous mail.

--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com