From: | jian he <jian(dot)universality(at)gmail(dot)com> |
---|---|
To: | John Naylor <johncnaylorls(at)gmail(dot)com> |
Cc: | Alexander Korotkov <aekorotkov(at)gmail(dot)com>, Steve Chavez <steve(at)supabase(dot)io>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Add pg_basetype() function to obtain a DOMAIN base type |
Date: | 2024-01-02 00:00:00 |
Message-ID: | CACJufxGGe8XK0gpQ3fC2kaVF794zeaq1u15JRcMW0CE=Mnbjhg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Dec 4, 2023 at 5:11 PM John Naylor <johncnaylorls(at)gmail(dot)com> wrote:
>
> On Thu, Sep 28, 2023 at 12:22 AM Alexander Korotkov
> <aekorotkov(at)gmail(dot)com> wrote:
> > The one thing triggering my perfectionism is that the patch does two
> > syscache lookups instead of one.
>
> For an admin function used interactively, I'm not sure why that
> matters? Or do you see another use case?
I did a minor refactor based on v1-0001.
I think pg_basetype should stay at "9.26.4. System Catalog Information
Functions".
So I placed it before pg_char_to_encoding.
Now functions listed on "Table 9.73. System Catalog Information
Functions" will look like alphabetical ordering.
I slightly changed the src/include/catalog/pg_proc.dat.
now it looks like very similar to pg_typeof
src6=# \df pg_typeof
List of functions
Schema | Name | Result data type | Argument data types | Type
------------+-----------+------------------+---------------------+------
pg_catalog | pg_typeof | regtype | "any" | func
(1 row)
src6=# \df pg_basetype
List of functions
Schema | Name | Result data type | Argument data types | Type
------------+-------------+------------------+---------------------+------
pg_catalog | pg_basetype | regtype | "any" | func
(1 row)
v2-0001 is as is in the first email thread, 0002 is my changes based on v2-0001.
Attachment | Content-Type | Size |
---|---|---|
v2-0001-Add-pg_basetype-regtype.patch | application/x-patch | 5.5 KB |
v2-0002-minor-refactor-based-on-v1-0001.patch | application/x-patch | 3.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Paul Jungwirth | 2024-01-02 01:59:47 | Re: SQL:2011 application time |
Previous Message | jian he | 2024-01-01 23:55:00 | Re: Change GUC hashtable to use simplehash? |