Re: [PATCH] Microvacuum for gist. - Mailing list pgsql-hackers

From Alexander Korotkov
Subject Re: [PATCH] Microvacuum for gist.
Date
Msg-id CAPpHfdv_ZpFYsEjC6zha7H6XBQ4UTnSFjNM7m-0+pyxLWrXePA@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Microvacuum for gist.  (Anastasia Lubennikova <a.lubennikova@postgrespro.ru>)
Responses Re: [PATCH] Microvacuum for gist.  (Anastasia Lubennikova <a.lubennikova@postgrespro.ru>)
List pgsql-hackers
On Mon, Aug 3, 2015 at 12:27 PM, Anastasia Lubennikova <a.lubennikova@postgrespro.ru> wrote:
1) Test and results are in attachments. Everything seems to work as expected.
2) I dropped these notices. It was done only for debug purposes. Updated patch is attached.
3) fixed

Good! Another couple of notes from me:
1) I think gistvacuumpage() and gistkillitems() need function-level comments.
2) ItemIdIsDead() can be used in index scan like it's done in _bt_checkkeys().

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
 

pgsql-hackers by date:

Previous
From: Anastasia Lubennikova
Date:
Subject: Re: [PATCH] Microvacuum for gist.
Next
From: Nikolay Shaplov
Date:
Subject: Re: pageinspect patch, for showing tuple data