=?utf-8?Q?=C3=81lvaro?= Herrera <alvhe...@alvh.no-ip.org> writes:
> Grumble. I'd rather introduce a partial restriction now only for names
> that affect the tools failing outright (pg_dumpall in particular), than
> do nothing for this release. If we feel the need to extend the
> restriction later, that's easy to do and bothers nobody. We've wanted
> these characters to be forbidden on database names for a long time, but
> nobody has cared as much for their use on other types of names. I'm not
> even sure we'd support the idea of forbidding them on all names (though
> the SQL standard doesn't allow control chars in identifiers.)
I'd be 100% behind forbidding all ASCII control characters in all
identifiers. I can't see any situation in which that's a good thing,
and I can think of plenty where it's a mistake (eg your editor
decided to change space to tab) or done with underhanded intent.
If we can cite the SQL standard then it's an entirely defensible
restriction.
Having said that, I'm not quite sure where we ought to implement
the restriction, and it's possible that there are multiple places
that would need to check. I concur that the day before feature
freeze is not a good time to be designing this. Let's defer.
regards, tom lane
