Comments on: Drupal + PostgreSQL: review some patches, folks!

By: catch

catch — Sun, 03 Aug 2008 09:13:04 +0000

@mike We do have a database abstraction layer planned - but as usual, progress has been held up by lack of postgres review:
http://drupal.org/node/225450

It should be ready to go now - just needs an install and the tests running through, so jump in!

By: Brenda

Brenda — Tue, 15 Jul 2008 04:55:48 +0000

There’s now a full set of regression unit tests for Drupal 7 (in cvs) — and they need to be run before something is commited

What we need now is folks running these tests against postgresql.

oh, and you’re welcome for the path a bunch of my postgres fixes snuck into the official tarball in this week security release.

By: selena

selena — Wed, 18 Jun 2008 03:45:08 +0000

@Mike - yeah, you noticed that too?

@Robert - I’ve met a few people, including Acquians and Lullabots. Maybe some social pressure on them to step it up? looks like there’s 40 people in the Pg group now. Might be enough to spur some action.

By: Robert Treat

Robert Treat — Wed, 18 Jun 2008 01:41:14 +0000

ISTM the real problem here wasn’t a lack of reviewers; that patch had been reviewed a field tested aplenty; but a problem of postgres’ continued treatment as a second class citizens by the drupal commiters, which lead to the problem having to be fixed several times more than necessary

By: Mike

Mike — Tue, 17 Jun 2008 12:19:06 +0000

Wow, that’s some ugly code Drupal is using there. Rewriting the SQL on the fly using regular expressions in PHP?! I hope they have some sort of abstraction layer planned for the future.