Re: [PATCH] Add min() and max() aggregate functions for xid8 - Mailing list pgsql-hackers

From Kyotaro Horiguchi
Subject Re: [PATCH] Add min() and max() aggregate functions for xid8
Date
Msg-id 20220209.120451.2033764091152450410.horikyota.ntt@gmail.com
Whole thread Raw
In response to Re: [PATCH] Add min() and max() aggregate functions for xid8  (Fujii Masao <masao.fujii@oss.nttdata.com>)
Responses Re: [PATCH] Add min() and max() aggregate functions for xid8  (Kyotaro Horiguchi <horikyota.ntt@gmail.com>)
List pgsql-hackers
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.

regards.

-- 
Kyotaro Horiguchi
NTT Open Source Software Center



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Plug minor memleak in pg_dump
Next
From: Robert Haas
Date:
Subject: Re: [PATCH v2] use has_privs_for_role for predefined roles