Re: tableam: abstracting relation sizing code

Поиск
Список
Период
Сортировка
От Daniel Gustafsson
Тема Re: tableam: abstracting relation sizing code
Дата
Msg-id EE6EBBB3-62E9-49D1-80AE-65C943983961@yesql.se
обсуждение исходный текст
Ответ на Re: tableam: abstracting relation sizing code  (Robert Haas <robertmhaas@gmail.com>)
Список pgsql-hackers
> On 10 Jun 2019, at 21:35, Robert Haas <robertmhaas@gmail.com> wrote:
>
> On Fri, Jun 7, 2019 at 6:42 PM Daniel Gustafsson <daniel@yesql.se> wrote:
>>> Good catch, and now I notice that the callback is not called
>>> estimate_rel_size but relation_estimate_size.  Updated patch attached;
>>> thanks for the review.
>>
>> The commit message still refers to it as estimate_rel_size though. The comment on
>> table_block_relation_estimate_size does too but that one might be intentional.
>
> Oops.  New version attached, hopefully fixing those and the compiler
> warning Alvaro noted.

+1 on this version of the patch, no warning, passes tests and looks good.

cheers ./daniel


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

Предыдущее
От: Tomas Vondra
Дата:
Сообщение: Re: Adaptive query optimization
Следующее
От: Tomas Vondra
Дата:
Сообщение: Re: [PATCH] Speedup truncates of relation forks