From: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Justin Pryzby <pryzby(at)telsasoft(dot)com> |
Cc: | "Andrey M(dot) Borodin" <x4mmm(at)yandex-team(dot)ru>, Darafei Komяpa Praliaskouski <me(at)komzpa(dot)net>, Pavel Borisov <pashkin(dot)elfe(at)gmail(dot)com>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Yet another fast GiST build |
Date: | 2020-09-21 11:56:44 |
Message-ID: | 075a990d-5a0f-1f62-8914-9ff13924e152@iki.fi |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 21/09/2020 02:06, Tom Lane wrote:
> Justin Pryzby <pryzby(at)telsasoft(dot)com> writes:
>> This also appears to break checksums.
Fixed, thanks for the report!
> I was wondering about that, because the typical pattern for use of
> smgrextend for indexes seems to be
>
> RelationOpenSmgr(rel);
> PageSetChecksumInplace(page, lastblock);
> smgrextend(rel->rd_smgr, MAIN_FORKNUM, lastblock, zerobuf.data, false);
>
> and gist_indexsortbuild wasn't doing either of the first two things.
>
> gist_indexsortbuild_flush_ready_pages looks like it might be
> a few bricks shy of a load too. But my local CLOBBER_CACHE_ALWAYS
> run hasn't gotten to anything except the pretty-trivial index
> made in point.sql, so I don't have evidence about it.
I added a RelationOpenSmgr() call there too, although it's not needed
currently. It seems to be enough to do it before the first smgrextend()
call. But if you removed or refactored the first call someohow, so it
was not the first call anymore, it would be easy to miss that you'd
still need the RelationOpenSmgr() call there. It's more consistent with
the code in nbtsort.c now, too.
- Heikki
From | Date | Subject | |
---|---|---|---|
Next Message | Dilip Kumar | 2020-09-21 12:03:43 | Re: Logical replication from PG v13 and below to PG v14 (devel version) is not working. |
Previous Message | Daniel Gustafsson | 2020-09-21 11:56:00 | Lift line-length limit for pg_service.conf |