HHVM-196: Manager::getServers() and ::debugInfo() should use mongoc_client_get_server_descriptions()

Author: derickr

bson.cpp

@@ -18,6 +18,7 @@
 #include "hphp/runtime/base/array-iterator.h"
 #include "hphp/runtime/base/execution-context.h"
 #include "hphp/runtime/base/type-string.h"
+#include "hphp/util/logger.h"
 
 #include "bson.h"
 #include "utils.h"

src/MongoDB/Driver/Manager.cpp

@@ -466,12 +466,18 @@ const StaticString
 Array HHVM_METHOD(MongoDBDriverManager, __debugInfo)
 {
 	MongoDBDriverManagerData* data = Native::data<MongoDBDriverManagerData>(this_);
+	size_t                        i;
+	mongoc_server_description_t **sds;
+	size_t                        n;
 	Array retval = Array::Create();
 	Array servers = Array::Create();
 
 	retval.add(s_MongoDBDriverManager_uri, (char*) mongoc_uri_get_string(data->m_client->uri));
 
-	mongoc_set_for_each(data->m_client->topology->description.servers, mongodb_driver_add_server_debug_wrapper, &servers);
+	sds = mongoc_client_get_server_descriptions(data->m_client, &n);
+	for (i = 0; i < n; i++) {
+		mongodb_driver_add_server_debug_wrapper(sds[i], &servers);
+	}
 
 	retval.add(s_MongoDBDriverManager_cluster, servers);
 
@@ -568,21 +574,20 @@ Array HHVM_METHOD(MongoDBDriverManager, getServers)
 {
 	MongoDBDriverManagerData *data = Native::data<MongoDBDriverManagerData>(this_);
 	size_t                    i;
-	mongoc_set_t             *set;
+	mongoc_server_description_t **sds;
+	size_t                         n;
 
 	Array retval = Array::Create();
 
-	set = data->m_client->topology->description.servers;
-	for (i = 0; i < set->items_len; i++) {
-		mongoc_server_description_t *sd = (mongoc_server_description_t*) set->items[i].item;
-
-		if (sd->type == MONGOC_SERVER_UNKNOWN) {
+	sds = mongoc_client_get_server_descriptions(data->m_client, &n);
+	for (i = 0; i < n; i++) {
+		if (sds[i]->type == MONGOC_SERVER_UNKNOWN) {
 			continue;
 		}
 
 		retval.add(
 			(int64_t) i,
-			hippo_mongo_driver_server_create_from_id(data->m_client, sd->id)
+			hippo_mongo_driver_server_create_from_id(data->m_client, sds[i]->id)
 		);
 
 	}

tests/MongoDBDriverManager_debugInfo.phpt

@@ -0,0 +1,62 @@
+--TEST--
+MongoDB\Driver\Manager::__debugInfo()
+--FILE--
+<?php
+$manager = new MongoDB\Driver\Manager();
+
+$command = new MongoDB\Driver\Command(array('ping' => 1));
+$manager->executeCommand("test", $command);
+
+var_dump( $manager->__debugInfo() );
+?>
+--EXPECTF--
+array(2) {
+  ["uri"]=>
+  string(%d) "%s"
+  ["cluster"]=>
+  array(1) {
+    [0]=>
+    array(10) {
+      ["host"]=>
+      string(%d) "%s"
+      ["port"]=>
+      int(%d)
+      ["type"]=>
+      int(1)
+      ["is_primary"]=>
+      bool(false)
+      ["is_secondary"]=>
+      bool(false)
+      ["is_arbiter"]=>
+      bool(false)
+      ["is_hidden"]=>
+      bool(false)
+      ["is_passive"]=>
+      bool(false)
+      ["last_is_master"]=>
+      array(8) {
+        ["ismaster"]=>
+        bool(true)
+        ["maxBsonObjectSize"]=>
+        int(%d)
+        ["maxMessageSizeBytes"]=>
+        int(%d)
+        ["maxWriteBatchSize"]=>
+        int(%d)
+        ["localTime"]=>
+        object(MongoDB\BSON\UTCDateTime)#4 (1) {
+          ["milliseconds"]=>
+          int(%d)
+        }
+        ["maxWireVersion"]=>
+        int(%d)
+        ["minWireVersion"]=>
+        int(0)
+        ["ok"]=>
+        float(1)
+      }
+      ["round_trip_time"]=>
+      int(%d)
+    }
+  }
+}

tests/MongoDBDriverManager_getServers.phpt

@@ -0,0 +1,36 @@
+--TEST--
+MongoDB\Driver\Manager::getServers()
+--FILE--
+<?php
+function assertServerType($type) {
+    if ($type === MongoDB\Driver\Server::TYPE_STANDALONE) {
+        printf("Found standalone server type: %d\n", $type);
+    } else {
+        printf("Unexpected server type: %d\n", $type);
+    }
+}
+
+$manager = new MongoDB\Driver\Manager();
+
+$servers = $manager->getServers();
+printf("Known servers: %d\n", count($servers));
+
+echo "Pinging\n";
+$command = new MongoDB\Driver\Command(array('ping' => 1));
+$manager->executeCommand("test", $command);
+
+$servers = $manager->getServers();
+printf("Known servers: %d\n", count($servers));
+
+foreach ($servers as $server) {
+    printf("Found server: %s:%d\n", $server->getHost(), $server->getPort());
+    assertServerType($server->getType());
+}
+
+?>
+--EXPECTF--
+Known servers: 0
+Pinging
+Known servers: 1
+Found server: %s:%d
+Found standalone server type: 1