Re: [PATCH] Microvacuum for gist.

Поиск
Список
Период
Сортировка
От Alexander Korotkov
Тема Re: [PATCH] Microvacuum for gist.
Дата
Msg-id CAPpHfdvp+NO-+1OeVVaLQQcEwJ-J-sFvrm6eNeXGLPZKeU5sRA@mail.gmail.com
обсуждение исходный текст
Ответ на [PATCH] Microvacuum for gist.  (Anastasia Lubennikova <lubennikovaav@gmail.com>)
Ответы Re: [PATCH] Microvacuum for gist.  (Jim Nasby <Jim.Nasby@BlueTreble.com>)
Re: [PATCH] Microvacuum for gist.  (Anastasia Lubennikova <a.lubennikova@postgrespro.ru>)
Список pgsql-hackers
Дерево обсуждения
Hi!

On Thu, Jul 30, 2015 at 2:51 PM, Anastasia Lubennikova <lubennikovaav@gmail.com> wrote:
I have written microvacuum support for gist access method.
Briefly microvacuum includes two steps:
1. When search tells us that the tuple is invisible to all transactions it is marked LP_DEAD and page is marked as "has dead tuples",
2. Then, when insert touches full page which has dead tuples it calls microvacuum instead of splitting page.
You can find a kind of review here [1].

Patch is in attachements. Please review it.

Nice!

Some notes about this patch.

1) Could you give same test case demonstrating that microvacuum really work with patch? Finally, we should get index less growing with microvacuum.

2) Generating notices for every dead tuple would be too noisy. I suggest to replace notice with one of debug levels.

+  elog(NOTICE, "gistkillitems. Mark Item Dead offnum %hd, blkno %d", offnum, BufferGetBlockNumber(buffer));


3) Please, recheck coding style. For instance, this line needs more spaces and open brace should be on the next line.

+ if ((scan->kill_prior_tuple)&&(so->curPageData > 0)&&(so->curPageData == so->nPageData)) {

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

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Kouhei Kaigai
Дата:
Сообщение: [BUG] CustomScan->custom_plans are not copied
Следующее
От: Tomas Vondra
Дата:
Сообщение: Re: multivariate statistics / patch v7