mongodb-ops 0.7.0 → 0.8.0

package/lib/mongodb-ops.js

@@ -33,35 +33,40 @@ class MongoDBOps {
   getObjectId(id) { return new ObjectID(id); }
 
   /**
-   * Static method - Open database connection
+   * Static method - Get DB client - It supports multiple db client with different connection string. Active db client will be reused for better performance
    * 
    * @param {string} connString Database connection string
    * @param {number} [poolSize] Database client pool size
-   * @returns {promise}
+   * @returns {promise} Promise with db client
    */
-  static async openDBConn(connString, poolSize = DEFAULT_POOLSIZE) {
-    // dbClient will be at static level for reusability
-    //
-    if (MongoDBOps.dbClient && MongoDBOps.dbClient.topology.s.state === "connected") { return Promise.resolve(); }
+  static async getDbClient(connString, poolSize = DEFAULT_POOLSIZE) {
     if (!connString) { throw new Error("missing-connection-string"); }
 
-    MongoDBOps.dbClient = await MongoClient.connect(connString, { 
+    if (!Array.isArray(MongoDBOps.dbClientList)) { MongoDBOps.dbClientList = []; }
+
+    for (let item of MongoDBOps.dbClientList) {
+      if (item.s.url === connString) {
+        if (item.topology.s.state !== "connected") {
+          item = await MongoClient.connect(connString, { 
+            poolSize: poolSize !== DEFAULT_POOLSIZE ? poolSize : MongoDBOps.poolSize || DEFAULT_POOLSIZE,
+            useNewUrlParser: true, 
+            useUnifiedTopology: true,
+          });
+        }
+
+        return Promise.resolve(item);
+      }
+    }
+
+    const newDbClient = await MongoClient.connect(connString, { 
       poolSize: poolSize !== DEFAULT_POOLSIZE ? poolSize : MongoDBOps.poolSize || DEFAULT_POOLSIZE,
       useNewUrlParser: true, 
       useUnifiedTopology: true,
     });
 
-    return Promise.resolve();
-  }
+    MongoDBOps.dbClientList.push(newDbClient);
 
-  /**
-   * Instance method - Open database connection
-   * 
-   * @param {number} [poolSize] Database client pool size
-   * @returns {promise}
-   */
-  async openDBConn(poolSize = DEFAULT_POOLSIZE) {
-    return Promise.resolve(await MongoDBOps.openDBConn(this.connString, poolSize));
+    return Promise.resolve(newDbClient);
   }
 
   /**
@@ -69,7 +74,11 @@ class MongoDBOps {
    * @returns {promise}
    */
   static async closeDBConn() {
-    if (MongoDBOps.dbClient) { await MongoDBOps.dbClient.close(); }
+    if (Array.isArray(MongoDBOps.dbClientList)) {
+      for (const item of MongoDBOps.dbClientList) {
+        await item.close();
+      }
+    }
     return Promise.resolve();
   }
 
@@ -78,8 +87,7 @@ class MongoDBOps {
    * @returns {promise}
    */
   async closeDBConn() {
-    if (MongoDBOps.dbClient) { await MongoDBOps.dbClient.close(); }
-    return Promise.resolve();
+    return Promise.resolve(await MongoDBOps.closeDBConn());
   }
 
   /**
@@ -97,10 +105,10 @@ class MongoDBOps {
    * @param {string} connString Database connection string
    * @returns {promise} Promise with object array
    */
-  static async getData(collectionName, queryExp, isAggregate = false, projection, sort, pagination, isGetCount = false, connString) {
-    if (!MongoDBOps.dbClient || (MongoDBOps.dbClient && MongoDBOps.dbClient.topology.s.state !== "connected")) { await MongoDBOps.openDBConn(connString); }   // reopen the db connection if it is not connected
-    let docs, db = MongoDBOps.dbClient.db();  // Create a new Db instance sharing the current socket connections
+  static async getData(collectionName, queryExp, isAggregate = false, projection, sort, pagination, isGetCount = false, connString) {   
+    const db = (await MongoDBOps.getDbClient(connString)).db();
 
+    let docs;
     if (isAggregate) { docs = await db.collection(collectionName).aggregate(queryExp).toArray(); }
     else {
       queryExp = queryExp || {};
@@ -162,11 +170,10 @@ class MongoDBOps {
    * @returns {promise}
    */
   static async writeData(type, collectionName, doc, filter, connString) {
-    if (!MongoDBOps.dbClient || (MongoDBOps.dbClient && MongoDBOps.dbClient.topology.s.state !== "connected")) { await MongoDBOps.openDBConn(connString); }   // reopen the db connection if it is not connected
-
     try {
-      let result, db = MongoDBOps.dbClient.db();  // Create a new Db instance sharing the current socket connections
+      const db = (await MongoDBOps.getDbClient(connString)).db();
 
+      let result;
       switch(type) {
         case "insertOne": result = await db.collection(collectionName).insertOne(doc); break;
         case "replaceOne": result = await db.collection(collectionName).replaceOne(filter, doc); break;
@@ -217,11 +224,10 @@ class MongoDBOps {
    * @returns {promise}
    */
   static async writeBulkData(type, collectionName, docs, ordered = false, connString) {
-    if (!MongoDBOps.dbClient || (MongoDBOps.dbClient && MongoDBOps.dbClient.topology.s.state !== "connected")) { await MongoDBOps.openDBConn(connString); }   // reopen the db connection if it is not connected
-
     try {
-      let result, db = MongoDBOps.dbClient.db();  // Create a new Db instance sharing the current socket connections
+      const db = (await MongoDBOps.getDbClient(connString)).db();
 
+      let result;
       switch(type) {
         case "insertBulk":
           for (let i = 0; i < docs.length; ++i) { docs[i] = { insertOne: { "document": docs[i] }}; }
@@ -247,13 +253,6 @@ class MongoDBOps {
           // }
           for (let i = 0; i < docs.length; ++i) { docs[i] = { updateOne: docs[i] }; }
           break;
-        case "deleteBulk":
-          // doc = {
-          //   "filter": <document>,
-          //   "collation": <document>
-          // }
-          for (let i = 0; i < docs.length; ++i) { docs[i] = { deleteOne: docs[i] }; }
-          break;
         case "allBulk":
           // allowed bulkWrite operations include insertOne, replaceOne, updateOne, updateMany, deleteOne, deleteMany
           break;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mongodb-ops",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "description": "Read and write ops for MongoDB",
   "main": "index.js",
   "scripts": {