Re: Combining Aggregates

Поиск
Список
Период
Сортировка
От Tomas Vondra
Тема Re: Combining Aggregates
Дата
Msg-id 12db0ced-2257-baf3-a97a-03d50a19d2de@2ndquadrant.com
обсуждение исходный текст
Ответ на Re: Combining Aggregates  (David Rowley <david.rowley@2ndquadrant.com>)
Ответы Re: Combining Aggregates  (Haribabu Kommi <kommi.haribabu@gmail.com>)
Список pgsql-hackers
Hi,

On 03/17/2016 12:53 PM, David Rowley wrote:>
...>
> I just had a quick skim over the patch and noticed the naming
> convention you're using for the combine function is *_pl, and you have
> float8_pl. There's already a function named float8pl() which is quite
> close to what you have. I've been sticking to *_combine() for these,
> so maybe float8_combine() and float8_regr_combine() are better names.

+1 to the _combine naming convention.

regards

-- 
Tomas Vondra                  http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



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

Предыдущее
От: Dmitry Dolgov
Дата:
Сообщение: Re: [PATH] Jsonb, insert a new value into an array at arbitrary position
Следующее
От: Alexander Kuleshov
Дата:
Сообщение: initdb: introduce PG_CMD_PRINTF intestead of PG_CMD_PRINTF{1,2,3}