On Thu, Sep 18, 2008 at 05:41:16PM +0300, Heikki Linnakangas wrote: > Attached is an updated version of the stripped-down patch. I've cleaned > it up a bit, and added more sanity checks. Documentation is still > missing and I haven't test it much.
FWIW, I disagree with the stripping down and think we should aim for the whole patch as submitted. Historically, the hardest part of getting collation support into postgres has been the catalog changes and easier the support for something other than OS locales. I supported the patch as it was precisely because it finally did that, and the stripping down takes us back to square one. Implementing the COLLATE keyword is much easier once you have the underlying support and woldn't involve any backward incompatabilities. The stripped down version gets us to a state which will only be supported (hopefully) for one release. For anyone counting, Firebird added support for ICU more than three years ago. Have a nice day, -- Martijn van Oosterhout <[EMAIL PROTECTED]> http://svana.org/kleptog/ > Please line up in a tree and maintain the heap invariant while > boarding. Thank you for flying nlogn airlines.
signature.asc
Description: Digital signature
