On Thu, Nov 10, 2016 at 4:33 PM, Michael Paquier <michael.paqu...@gmail.com> wrote: > On Thu, Nov 10, 2016 at 4:23 PM, konstantin knizhnik > <k.knizh...@postgrespro.ru> wrote: >> No, it is latest sources from Postgres repository. >> Please notice that you should create new database and tablespace to >> reproduce this issue. >> So actually the whole sequence is >> >> mkdir fs >> initdb -D pgsql >> pg_ctl -D pgsql -l logfile start >> psql postgres >> # create tablespace fs location '/home/knizhnik/dtm-data/fs'; >> # set default_tablespace=fs; >> # create unlogged table foo(x integer); >> # insert into foo values(generate_series(1,100000)); >> # ^D >> pkill -9 postgres >> pg_ctl -D pgsql -l logfile start >> # select * from foo; > > OK, I understood what I was missing. This can be reproduced with > wal_level = minimal. When using hot_standby things are working > properly.
Okay, so what happens is that the CREATE TABLESPACE record removes the tablespace directory and recreates a fresh one, but as no CREATE records are created for unlogged tables the init fork is not re-created. It seems to me that we should log a record to recreate the INIT fork, and that heap_create_with_catalog() is missing something. Generating a record in RelationCreateStorage() is the more direct way, and that actually fixes the issue. Now the comments at the top of it mention that RelationCreateStorage() should only be used to create the MAIN forknum. So, wouldn't a correct fix be to log this INIT record at the end of heap_create()? -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers