Re: Minor comment update in execPartition.c

Поиск
Список
Период
Сортировка
От Etsuro Fujita
Тема Re: Minor comment update in execPartition.c
Дата
Msg-id 5ADFF13F.9070409@lab.ntt.co.jp
обсуждение исходный текст
Ответ на Re: Minor comment update in execPartition.c  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
Список pgsql-hackers
(2018/04/25 11:05), Alvaro Herrera wrote:
> Amit Langote wrote:
>
>>> I think we should remove the words "if not already done" from that
>>> comment because 1) that function is called if the partition wasn't
>>> already initialized and 2) that function assumes that.  Attached is a
>>> small patch for removing the words.
>>
>> Thanks, sounds fine.  I think those words remain from earlier versions of
>> the patch committed as edd44738bc8 [1], where there used to be a fast-path
>> return if the ResultRelInfo was already non-NULL for the passed in index.
>
> Makes sense -- pushed, thanks both.

Thanks for committing, Alvaro!  Thanks for reviewing, Amit!

Best regards,
Etsuro Fujita


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

Предыдущее
От: Amit Langote
Дата:
Сообщение: Re: Oddity in tuple routing for foreign partitions
Следующее
От: Christophe Pettus
Дата:
Сообщение: Re: Built-in connection pooling