On Wed, Oct 22, 2025 at 5:32 PM Tim Waizenegger <[email protected]> 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.
v2-0001-refactor-pg_get_domain_ddl.no-cfbot
Description: Binary data
