From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
---|---|
To: | a(dot)kozhemyakin(at)postgrespro(dot)ru |
Cc: | Amul Sul <sulamul(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, thomas(dot)munro(at)gmail(dot)com, tgl(at)sss(dot)pgh(dot)pa(dot)us, noah(at)leadboat(dot)com |
Subject: | Re: tweak to a few index tests to hits ambuildempty() routine. |
Date: | 2022-09-21 11:58:37 |
Message-ID: | 20220921115837.ob243hlokrlrxvfo@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2022-Sep-21, a(dot)kozhemyakin(at)postgrespro(dot)ru wrote:
> After analyzing this, I found out why we don't reach that Assert but we have
> coverage shown - firstly, it reached via another test, vacuum; secondly, it
> depends on the gcc optimization flag. We reach that Assert only when using
> -O0.
> If we build with -O2 or -Og that function is not reached (due to different
> results of the heap_prune_satisfies_vacuum() check inside
> heap_page_prune()).
> But as the make checks mostly (including the buildfarm testing) performed
> with -O2/-Og, it looks like that after 4fb5c794e5 we have lost the coverage
> provided by the 4c51a2d1e4.
Hmm, so if we merely revert the change to gin.sql then we still won't
get the coverage back? I was thinking that a simple change would be to
revert the change from temp to unlogged for that table, and create
another unlogged table; but maybe that's not enough. Do we need a
better test for GIN vacuuming that works regardless of the optimization
level?
--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
"Investigación es lo que hago cuando no sé lo que estoy haciendo"
(Wernher von Braun)
From | Date | Subject | |
---|---|---|---|
Next Message | Damir Belyalov | 2022-09-21 12:11:46 | Re: POC PATCH: copy from ... exceptions to: (was Re: VLDB Features) |
Previous Message | kuroda.hayato@fujitsu.com | 2022-09-21 11:56:56 | RE: [Proposal] Add foreign-server health checks infrastructure |