Re: [PATCH] pg_get_domain_ddl: DDL reconstruction function for CREATE DOMAIN statement
jian he <jian.universality@gmail.com>
From: jian he <jian.universality@gmail.com>
To: Tim Waizenegger <tim.waizenegger@enterprisedb.com>
Cc: pgsql-hackers@postgresql.org
Date: 2025-10-23T04:21:34Z
Lists: pgsql-hackers
Attachments
- v2-0001-refactor-pg_get_domain_ddl.no-cfbot (application/octet-stream)
On Wed, Oct 22, 2025 at 5:32 PM Tim Waizenegger
<tim.waizenegger@enterprisedb.com> wrote:
>
> updated patch is attached
>
I’ve done some refactoring, hope it’s now more intuitive to you.
Since a domain’s base type can itself be another domain, it’s better to use
appendStringInfo(&buf, "CREATE DOMAIN %s AS %s",
generate_qualified_type_name(domain_oid),
generate_qualified_type_name(typForm->typbasetype));
then the domain's base type is also fully qualified.
I also refactored the logic for printing domain constraints, which should reduce
syscache lookups or table scans compared to your version.
please check the attached.