Re: pg_type.h regression? - Mailing list pgsql-hackers

From Tom Lane
Subject Re: pg_type.h regression?
Date
Msg-id 27694.1222178193@sss.pgh.pa.us
Whole thread Raw
In response to Re: pg_type.h regression?  ("Greg Sabino Mullane" <greg@turnstep.com>)
Responses Re: pg_type.h regression?  ("Greg Sabino Mullane" <greg@turnstep.com>)
List pgsql-hackers
"Greg Sabino Mullane" <greg@turnstep.com> writes:
>> I think my reasoning was that all array types should have typdelim = ','
>> for consistency.  It doesn't actually matter because nothing looks at
>> the value ... the element type's delimiter is what array_in/out use.

> Ah, okay, that makes sense. Thanks for the explanation, I'll tweak
> my code to do it the right way by looking at the base type.

Well, plan B is that we could redefine things as "all array types should
have typdelim equal to their base type's typdelim".  The backend still
wouldn't care, but if there are other clients out there confusing the
two, maybe this is important to worry about.

DefineType currently always sets the array's typdelim to ',', so I
figured that the built-in types should match that.  But we could
easily change both of them to do the other.  Thoughts?
        regards, tom lane


pgsql-hackers by date:

Previous
From: Stephen Frost
Date:
Subject: Re: Proposal: move column defaults into pg_attribute along with attacl
Next
From: Tom Lane
Date:
Subject: Re: Proposal: move column defaults into pg_attribute along with attacl