[development] Do not let postgresql hold back great patches

Earnest Berry III earnest.berry at gmail.com
Mon Nov 12 23:34:47 UTC 2007

I'm just catching up on this thread, so this may have already been answered.
Regarding the case insensitivy, why can't one use ILIKE during the postGres
search in the SQL statement to force case-insensitive?...again, as I read
more through this thread, this may have been answered.


From: development-bounces at drupal.org [mailto:development-bounces at drupal.org]
On Behalf Of Khalid Baheyeldin
Sent: Sunday, November 11, 2007 7:31 PM
To: development at drupal.org
Subject: Re: [development] Do not let postgresql hold back great patches


On Nov 11, 2007 5:17 PM, Karoly Negyesi <karoly at negyesi.net> wrote:

In my opinion, any patch should be committable to core once it works
on mysql and has a decent hope (use common sense) to work on
postgresql. Once it's committed those who care about postgresql, if
they so want to, can test and if there is a need, fix it. 

Prime example: http://drupal.org/node/146466 this is the most
important patch we have currently as it makes Drupal search speedy and
nice. And it is more or less on hold just because noone is sure 
whether the postgresql update works or not.

I know many will not like what I will say, but  I have to say it.

A prime example of where MySQL works fine to solve an issue with a few lines
is this issue http://drupal.org/node/83738. All the huge changes and jumping
through hoops is because of accomodating PostgreSQL 

We introduced schema changes just because PostgreSQL cannot do case
insensitive matching by default, while MySQL works fine. For the sake of 5%
(or 1%) of the sites, we are increasing complexity. 


Karoly Negyesi

Khalid M. Baheyeldin
Drupal optimization, development, customization and consulting. 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.drupal.org/pipermail/development/attachments/20071112/b3ca0653/attachment.htm 

More information about the development mailing list