Re: pgsql: Skip full index scan during cleanup of B-tree indexes whenpossi - Mailing list pgsql-committers

From Teodor Sigaev
Subject Re: pgsql: Skip full index scan during cleanup of B-tree indexes whenpossi
Date
Msg-id b8b92fda-e1b1-ba23-fca8-578a34a5e7f2@sigaev.ru
Whole thread Raw
In response to Re: pgsql: Skip full index scan during cleanup of B-tree indexes when possi  (Alexander Korotkov <a.korotkov@postgrespro.ru>)
Responses Re: pgsql: Skip full index scan during cleanup of B-tree indexes when possi  (Alexander Korotkov <a.korotkov@postgrespro.ru>)
List pgsql-committers
>     Another thing that I noticed is that the metapage stores
>     btm_last_cleanup_num_heap_tuples as a float4, even though
>     xl_btree_metadata stores it as a double. I suggest that both places
>     store it as float8, to be consistent. (It should not be double because
>     we always avoid using anything other types with explicit typedef'd
>     widths in WAL records.)
> 
> 
> Good catch, thank you!  I also agree that both these fields should be of 
> float8 type.
> Please, find attached bt-vacuum-cleanup-float8-num-heap-tuples.patch 
> fixing that.

Hm. patch changes on-disk representation of btree meta page. And there 
is no support to upgrade meta page since 857f9c36 commit (and should 
not, because that versions wasn't relesead), to prevent possible false 
positive bug reports I suggest to bump catversion. Objections?


-- 
Teodor Sigaev                      E-mail: teodor@sigaev.ru
                                       WWW: http://www.sigaev.ru/


pgsql-committers by date:

Previous
From: Teodor Sigaev
Date:
Subject: pgsql: Adjust INCLUDE index truncation comments and code.
Next
From: Teodor Sigaev
Date:
Subject: pgsql: Handle XLOG_BTREE_META_CLEANUP in btree_desc() andbtree_identif