Add support for the mongo 1.3 drivers and read perferences. Mongo 1.3 driver for php makes a number of changes including how slave operations are targetted and also makes all write operations write safe. This patch (which will appear shortly) loads the new mongoclient class if available and also updates all operations to be safe so as 1.3 and 1.2 drivers behaviour in the exact same way.

Files: 
CommentFileSizeAuthor
#7 mongo.1.3.patch24.11 KBmarcingy
Test request sent.
[ View ]
#6 1.3.patch23.89 KBmarcingy
Test request sent.
[ View ]
#5 mongo.1.3.patch22.8 KBmarcingy
Test request sent.
[ View ]
#3 mongo-1.3-r.patch21.24 KBmarcingy
Test request sent.
[ View ]
#2 mongo-1.3.d.o.patch20.06 KBmarcingy
Test request sent.
[ View ]
#1 mongo-1.3.d.o.patch20.15 KBmarcingy
Test request sent.
[ View ]

Comments

marcingy’s picture

Status:Active» Needs review
StatusFileSize
new20.15 KB
Test request sent.
[ View ]
marcingy’s picture

StatusFileSize
new20.06 KB
Test request sent.
[ View ]

Tweaks after speaking to chx

marcingy’s picture

StatusFileSize
new21.24 KB
Test request sent.
[ View ]

Reroll

Rok Žlender’s picture

Status:Needs review» Needs work
+++ b/README.txtundefined
@@ -189,6 +211,22 @@ the hypothetical connection alias 'logginghost'
+    db_connection => 'logginghost',

Missing single quotes around db_connection also db_conncetion can have the same options as without using read_preference maybe put that in as documentation, host, db..

mongodb_session/mongodb_session.module has an update you missed

drush_mongodb_migrate in mongodb_migrate/mongodb_migrate.drush.inc has an insert function that might need safe or w=0

marcingy’s picture

Status:Needs work» Needs review
StatusFileSize
new22.8 KB
Test request sent.
[ View ]

Tweaked

marcingy’s picture

StatusFileSize
new23.89 KB
Test request sent.
[ View ]

Re-roll, as today we uncovered an issue on our server where by attempts were being made to update a slave. This patch restricts the cron update to the connection the queue belongs too. In some ways it may need a refractoring.

marcingy’s picture

StatusFileSize
new24.11 KB
Test request sent.
[ View ]

Fixes up alias logic for backwards comptability

ilari.stenroth’s picture

Any news on progress of merging this patch to the project? It's urgent to get Drupal support mongo PHP extension 1.3+ API.

chx’s picture

Status:Needs review» Fixed

Committed, thanks.

Status:Fixed» Closed (fixed)

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

amontero’s picture

Issue summary:View changes

Modified existing issue to focus on RC3:
#2017091: Roadmap to RC3 release