Hi, sounds great, I have all my sites using postgres/postgis for advanced spatial uses. is PG in your roadmap?

Comments

andrewlevine’s picture

Hey, unfortunately I don't have enough time or Postgres chops. If someone came along and offered a ton of help it could be possible but otherwise I don't see it happening.

andrewlevine’s picture

Status: Active » Postponed
andrewlevine’s picture

Forgot to mention that I'm hoping it also might be possible in Drupal 7 with the new database API. I'd love to have postgres support but I just don't have resources to work on it at the moment.

pounard’s picture

I might have to do it, we use some Drupal over PostgreSQL installs, and I really don't like MySQL (it's such a shame to call it a DBMS).
If I have some hours to spend, I'll do it.

Not any promises here, just I'd like to see this feature too. Is there any kind of warning you should tell before doing this? Like, your db patch, is it MySQL only or should it exists for PostgreSQL layer too?

EDIT: I should have better looked at db patch, it's in common database.inc :)

pounard’s picture

I did it. It works on my environment.

I wrote this patch on a DataSync already patched version (I now, this is EVIL), so you have to pass this patch before: http://drupal.org/files/issues/536528_catch_exceptions3.patch

(nooo don't hit me!!!)

This code may need some review, andrewlevine please review it!

pounard’s picture

Status: Postponed » Needs review
pounard’s picture

Another note, for people which are running buggy linux distribution, such as Ubuntu, you'll have to follow this procedure to make php postgresql extension working in CLI mode:
http://geek.joshwaihi.com/content/php-cli-segmentation-fault-pgsql

andrewlevine’s picture

I'm fine with this patch, since it doesn't seem to affect anything with mysql. Have you tested this and confirmed that this is all it takes to get datasync running with postgres (albeit in an alpha state)?

pounard’s picture

I ran postgres developing my custom migration module during 3 days with this patch. 8hours/day, so yes, on my env it works.
EDIT: it still need testing from some outsiders

andrewlevine’s picture

Status: Needs review » Fixed

committed. thanks!

spydmobile’s picture

Wow folks! I am mid forest fire season right now, but as soon as things cool of and the rain starts I can test this! Thank you!!!!!

pounard’s picture

you're welcome :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.