Further comments welcome so I'm putting it back into needs review for the moment while I work on the refactor.
attached fixed patch
all tests passed
doc build without problems
Thanks.
Actually, one question I didn't pose before, does the SQL standard define DROP TYPE to target domains while also providing for a DROP DOMAIN command? Do drop commands for the other types we have not exist because those aren't SQL standard types (or the standard they are standard types but the commands aren't defined)?