Re: [PATCH] Add min() and max() aggregate functions for xid8

Поиск
Список
Период
Сортировка
От Fujii Masao
Тема Re: [PATCH] Add min() and max() aggregate functions for xid8
Дата
Msg-id 1a101cb1-7c9c-d018-4a85-694d708c244f@oss.nttdata.com
обсуждение исходный текст
Ответ на Re: [PATCH] Add min() and max() aggregate functions for xid8  (Kyotaro Horiguchi <horikyota.ntt@gmail.com>)
Список pgsql-hackers

On 2022/02/09 13:04, Kyotaro Horiguchi wrote:
> At Wed, 09 Feb 2022 12:04:51 +0900 (JST), Kyotaro Horiguchi <horikyota.ntt@gmail.com> wrote in
>> At Wed, 9 Feb 2022 11:01:57 +0900, Fujii Masao <masao.fujii@oss.nttdata.com> wrote in
>>> Agreed. So barring any objection, I will commit that patch.
>>
>> Sorry for being late, but I don't like the function names.
>>
>> +xid8_larger(PG_FUNCTION_ARGS)
>> +xid8_smaller(PG_FUNCTION_ARGS)
>>
>> Aren't they named like xid8gt and xid8lt conventionally?  At least the
>> name lacks a mention to equality.
> 
> Ah, sorry. the function returns larger/smaller one from the
> parameters. Sorry for the noise.

Thanks for the review! I pushed the patch.

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION



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

Предыдущее
От: Julien Rouhaud
Дата:
Сообщение: Re: Database-level collation version tracking
Следующее
От: "houzj.fnst@fujitsu.com"
Дата:
Сообщение: RE: row filtering for logical replication