version bump test

lib/storage/metadata/mongoclient/DataCounter.js

@@ -0,0 +1,256 @@
+const NEW_OBJ = 0;
+const NEW_VER = 1;
+const UPDATE_VER = 2;
+const UPDATE_MST = 3;
+const RESTORE = 4;
+
+const DEL_VER = 0;
+const DEL_MST = 1;
+
+const CURR = 'curr';
+const PREV = 'prev';
+
+function deepCopyObject(obj) {
+    return JSON.parse(JSON.stringify(obj));
+}
+
+class DataCounter {
+    /**
+     * DataCounter - class for keeping track of the ItemCount metrics
+     * @return {DataCounter} DataCounter object
+     */
+    constructor() {
+        this.objects = 0;
+        this.versions = 0;
+        this.buckets = 0;
+        this.bucketList = [];
+        this.dataManaged = {
+            total: { curr: 0, prev: 0 },
+            byLocation: {},
+        };
+        this.busy = false;
+        this.populated = false;
+    }
+
+    /**
+     * set - set DataCounter values
+     * @param {Object} setVal - object containing values to be used for setting
+     * DataCounter
+     * @param {number} setVal.objects - number of master objects
+     * @param {number} setVal.versions - number of versioned objects
+     * @param {number} setVal.buckets - number of buckets
+     * @param {string[]} setVal.bucketList - list of bucket names
+     * @param {Object} setVal.dataManaged - object containing information about
+     * all the data managed
+     * @param {Object} setVal.total - object containing the total byte count of
+     * data managed
+     * @param {number} setVal.total.curr - the total byte count of master
+     * objects
+     * @param {number} setVal.total.prev - the total byte count of versioned
+     * objects
+     * @param {Object} setVal.byLocaton - object containing the information
+     * about data managed on each location
+     * @return {undefined}
+     */
+    set(setVal) {
+        if (setVal) {
+            this.objects = setVal.objects;
+            this.versions = setVal.versions;
+            this.buckets = setVal.buckets;
+            this.bucketList = [...setVal.bucketList];
+            this.dataManaged = deepCopyObject(setVal.dataManaged);
+            this.busy = false;
+            this.populated = true;
+        }
+    }
+
+    /**
+     * results - creates a deep copy of the current DataCounter values
+     * @return {Object} - object containing the current DataCounter values
+     */
+    results() {
+        const obj = {
+            objects: this.objects,
+            versions: this.versions,
+            buckets: this.buckets,
+            bucketList: this.bucketList,
+            dataManaged: this.dataManaged,
+        };
+        return deepCopyObject(obj);
+    }
+
+    /**
+     * addObjectFn - performing add operations
+     * @param {ObjectMD} currMD - new master version metadata
+     * @param {ObjectMD} prevMD - old master version metadata
+     * @param {number} type - index of the current type of add operation
+     * @return {undefined}
+     */
+    addObject(currMD, prevMD, type) {
+        if (type !== undefined && type !== null && this.populated) {
+            switch (type) {
+            case NEW_OBJ: // add new object, replace master if needed
+                if (prevMD) {
+                    this._delValue(prevMD, CURR);
+                    this._addValue(currMD, CURR);
+                } else {
+                    ++this.objects;
+                    this._addValue(currMD, CURR);
+                }
+                break;
+            case NEW_VER: // add new object, archive master
+                ++this.versions;
+                this._delValue(prevMD, CURR);
+                this._addValue(prevMD, PREV);
+                this._addValue(currMD, CURR);
+                break;
+            case UPDATE_VER: // update archived object, replication info
+                this._updateObject(currMD, prevMD, PREV);
+                break;
+            case UPDATE_MST: // update master object, replication info
+                this._updateObject(currMD, prevMD, CURR);
+                break;
+            case RESTORE:
+                --this.versions;
+                this._delValue(currMD, PREV);
+                ++this.objects;
+                this._addValue(currMD, CURR);
+                break;
+            default:
+                // should throw error, noop
+                break;
+            }
+        }
+    }
+
+    /**
+     * delObjectFn - performing del operations
+     * @param {ObjectMD} currMD - object metadata
+     * @param {number} type - index of the current type of delete operation
+     * @return {undefined}
+     */
+    delObject(currMD, type) {
+        if (type !== undefined && type !== null && this.populated) {
+            switch (type) {
+            case DEL_VER:
+                --this.versions;
+                this._delValue(currMD, PREV);
+                break;
+            case DEL_MST:
+                --this.objects;
+                this._delValue(currMD, CURR);
+                break;
+            default:
+                // should throw error, noop
+                break;
+            }
+        }
+    }
+
+    _addLocation(site, size, type) {
+        this.dataManaged.total[type] += size;
+        if (!this.dataManaged.byLocation[site]) {
+            this.dataManaged.byLocation[site] = {
+                curr: 0,
+                prev: 0,
+            };
+        }
+        this.dataManaged.byLocation[site][type] += size;
+    }
+
+    /**
+     * _addValue - helper function for handling put object updates
+     * @param {ObjectMD} objMD - object metadata
+     * @param {string} type - string with value either 'curr' or 'prev'
+     * @return {undefined}
+     */
+    _addValue(objMD, type) {
+        if (objMD) {
+            const { replicationInfo, 'content-length': size } = objMD;
+            const { backends } = replicationInfo || {};
+            this._addLocation(objMD.dataStoreName, size, type);
+            if (backends && Array.isArray(backends)) {
+                backends.forEach(loc => {
+                    const { site, status } = loc;
+                    if (status === 'COMPLETED') {
+                        this._addLocation(site, size, type);
+                    }
+                });
+            }
+        }
+    }
+
+    /**
+     * _updateObject - helper function for handling updates from replication
+     * info changes
+     * @param {ObjectMD} currMD - new object metadata
+     * @param {ObjectMD} prevMD - old object metadata
+     * @param {string} type - string with value either 'curr' or 'prev'
+     * @return {undefined}
+     */
+    _updateObject(currMD, prevMD, type) {
+        if (currMD && prevMD) {
+            // check for changes in replication
+            const { replicationInfo: currLocs,
+                'content-length': size } = currMD;
+            const { replicationInfo: prevLocs } = prevMD;
+            const { backends: prevBackends } = prevLocs || {};
+            const { backends: currBackends } = currLocs || {};
+            const oldLocs = {};
+            if (prevBackends && Array.isArray(prevBackends)) {
+                prevBackends.forEach(loc => {
+                    const { site, status } = loc;
+                    oldLocs[site] = status;
+                });
+            }
+            if (currBackends && Array.isArray(currBackends)) {
+                currBackends.forEach(loc => {
+                    const { site, status } = loc;
+                    if (site in oldLocs && status === 'COMPLETED' &&
+                        oldLocs[site] !== status) {
+                        this._addLocation(site, size, type);
+                    }
+                });
+            }
+        }
+    }
+
+    _delLocation(site, size, type) {
+        if (this.dataManaged.byLocation[site]) {
+            this.dataManaged.total[type] -= size;
+            this.dataManaged.byLocation[site][type] -= size;
+        }
+    }
+    /**
+     * _delValue - helper function for handling delete object operations
+     * @param {ObjectMD} objMD - object metadata
+     * @param {string} type - string with value either 'curr' or 'prev'
+     * @return {undefined}
+     */
+    _delValue(objMD, type) {
+        if (objMD) {
+            const { replicationInfo, 'content-length': size } = objMD;
+            const { backends } = replicationInfo || {};
+            this._delLocation(objMD.dataStoreName, size, type);
+            if (backends && Array.isArray(backends)) {
+                backends.forEach(loc => {
+                    const { site, status } = loc;
+                    if (status === 'COMPLETED') {
+                        this._delLocation(site, size, type);
+                    }
+                });
+            }
+        }
+    }
+}
+
+module.exports = {
+    NEW_OBJ,
+    NEW_VER,
+    UPDATE_VER,
+    UPDATE_MST,
+    RESTORE,
+    DEL_VER,
+    DEL_MST,
+    DataCounter,
+};

lib/storage/metadata/mongoclient/MongoClientInterface.js

@@ -25,6 +25,16 @@ const listAlgos = require('../../../algos/list/exportAlgos');
 
 const MongoReadStream = require('./readStream');
 const MongoUtils = require('./utils');
+const {
+    NEW_OBJ,
+    NEW_VER,
+    UPDATE_VER,
+    UPDATE_MST,
+    RESTORE,
+    DEL_VER,
+    DEL_MST,
+    DataCounter,
+} = require('./DataCounter');
 
 const USERSBUCKET = '__usersbucket';
 const METASTORE = '__metastore';
@@ -65,6 +75,13 @@ function generatePHDVersion(versionId) {
     };
 }
 
+function isUserBucket(bucketName) {
+    return bucketName !== METASTORE &&
+        bucketName !== INFOSTORE &&
+        bucketName !== USERSBUCKET &&
+        bucketName !== PENSIEVE &&
+        !bucketName.startsWith(constants.mpuBucketPrefix);
+}
 
 /**
  * @constructor
@@ -94,7 +111,7 @@ class MongoClientInterface {
         this.replicationGroupId = replicationGroupId;
         this.database = database;
         this.lastItemScanTime = null;
-        this.lastItemScanResult = null;
+        this.dataCount = new DataCounter();
     }
 
     setup(cb) {
@@ -477,21 +494,107 @@ class MongoClientInterface {
         });
     }
 
+    _getPrevMD(c, params, isUpdate, log, cb) {
+        const collectionName = c.s.name;
+        const { objName, versionId } = params;
+        if (isUserBucket(collectionName)) {
+            if (isUpdate) {
+                return c.find({
+                    'value.versionId': versionId,
+                }, {}).toArray((err, entries) => {
+                    if (err) {
+                        log.error(err.message, {
+                            error: err,
+                        });
+                    }
+                    return cb(null, entries);
+                });
+            }
+            return c.findOne({
+                _id: objName,
+            }, {}, (err, preVal) => {
+                if (err) {
+                    log.error(err.message, {
+                        error: err,
+                    });
+                    return cb();
+                }
+                return cb(null, preVal);
+            });
+        }
+        return cb();
+    }
+
     putObject(bucketName, objName, objVal, params, log, cb) {
         MongoUtils.serialize(objVal);
         const c = this.getCollection(bucketName);
+        const countParams = {
+            objName,
+            versionId: params.versionId,
+        };
+        const isUpdate = params && params.versionId;
+        const isVer = params && ((params.versioning && !params.versionId) ||
+            params.versionId);
+
+        async.series({
+            prevMD: done =>
+                this._getPrevMD(c, countParams, isUpdate, log, done),
+            putObjectRes: done => {
                 if (params && params.versioning && !params.versionId) {
-            return this.putObjectVerCase1(c, bucketName, objName, objVal,
+                    return this.putObjectVerCase1(
-                                          params, log, cb);
+                        c, bucketName, objName, objVal, params, log, done);
                 } else if (params && params.versionId === '') {
-            return this.putObjectVerCase2(c, bucketName, objName, objVal,
+                    return this.putObjectVerCase2(
-                                          params, log, cb);
+                        c, bucketName, objName, objVal, params, log, done);
                 } else if (params && params.versionId) {
-            return this.putObjectVerCase3(c, bucketName, objName, objVal,
+                    return this.putObjectVerCase3(
-                                          params, log, cb);
+                        c, bucketName, objName, objVal, params, log, done);
                 }
-        return this.putObjectNoVer(c, bucketName, objName, objVal,
+                return this.putObjectNoVer(
-                                   params, log, cb);
+                    c, bucketName, objName, objVal, params, log, done);
+            },
+        }, (err, result) => {
+            if (err) {
+                return cb(err);
+            }
+            const { prevMD, putObjectRes } = result;
+            if (isUpdate) {
+                switch (prevMD.length) {
+                case 2:
+                    // length == 2 => master entry + version entry
+                    // updating master object MD
+                    this.dataCount.addObject(
+                        objVal, prevMD[0].value, UPDATE_MST);
+                    break;
+                case 1:
+                    // length == 1 => version entry
+                    // updating version object MD
+                    this.dataCount.addObject(
+                        objVal, prevMD[0].value, UPDATE_VER);
+                    break;
+                case 0:
+                    // changing null id to versioning compatible id
+                    // no data count changes
+                    break;
+                default:
+                    // something seems wrong if length is not 0, 1, or 2
+                    log.debug('invalid number of object entries');
+                    break;
+                }
+            } else {
+                if (!prevMD || !prevMD.value) {
+                    this.dataCount.addObject(
+                        objVal, null, NEW_OBJ);
+                } else if (isVer) {
+                    this.dataCount.addObject(
+                        objVal, prevMD.value, NEW_VER);
+                } else {
+                    this.dataCount.addObject(
+                        objVal, prevMD.value, NEW_OBJ);
+                }
+            }
+            return cb(null, putObjectRes);
+        });
     }
 
     getObject(bucketName, objName, params, log, cb) {
@@ -585,6 +688,9 @@ class MongoClientInterface {
                 { error: err.message });
                 return cb(errors.InternalError);
             }
+            if (isUserBucket(c.s.name)) {
+                this.dataCount.addObject(objVal, null, RESTORE);
+            }
             return cb(null);
         });
     }
@@ -742,10 +848,26 @@ class MongoClientInterface {
             if (mst.value.isPHD ||
                 mst.value.versionId === params.versionId) {
                 return this.deleteObjectVerMaster(c, bucketName, objName,
-                                                  params, log, cb);
+                params, log, err => {
+                    if (err) {
+                        return cb(err);
+                    }
+                    if (isUserBucket(c.s.name)) {
+                        this.dataCount.delObject(mst.value, DEL_MST);
+                    }
+                    return cb();
+                });
             }
             return this.deleteObjectVerNotMaster(c, bucketName, objName,
-                                                 params, log, cb);
+            params, log, err => {
+                if (err) {
+                    return cb(err);
+                }
+                if (isUserBucket(c.s.name)) {
+                    this.dataCount.delObject(mst.value, DEL_VER);
+                }
+                return cb();
+            });
         });
     }
 
@@ -768,6 +890,10 @@ class MongoClientInterface {
                     { error: err.message });
                 return cb(errors.InternalError);
             }
+            const objMD = result.value || {};
+            if (isUserBucket(c.s.name)) {
+                this.dataCount.delObject(objMD.value, DEL_MST);
+            }
             return cb(null);
         });
     }
@@ -894,10 +1020,10 @@ class MongoClientInterface {
     }
 
     countItems(log, cb) {
-        if (this.lastItemScanTime !== null &&
+        // if (this.lastItemScanTime !== null &&
-            (Date.now() - this.lastItemScanTime) <= itemScanRefreshDelay) {
+        //     (Date.now() - this.lastItemScanTime) <= itemScanRefreshDelay) {
-            return process.nextTick(cb, null, this.lastItemScanResult);
+        //     return process.nextTick(cb, null, this.lastItemScanResult);
-        }
+        // }
 
         const res = {
             objects: 0,
@@ -965,12 +1091,11 @@ class MongoClientInterface {
                     if (err) {
                         return next(errors.InternalError);
                     }
-                    if (results.bucketList && results.counter &&
+                    if (results.bucketList && results.dataManaged) {
-                        results.counter.dataManaged) {
                         res.bucketList.push(results.bucketList);
-                        res.objects += results.counter.objects;
+                        res.objects += results.objects;
-                        res.versions += results.counter.versions;
+                        res.versions += results.versions;
-                        consolidateData(results.counter.dataManaged);
+                        consolidateData(results.dataManaged);
                     }
                     return next();
                 });
@@ -979,173 +1104,62 @@ class MongoClientInterface {
                     return cb(err);
                 }
                 this.lastItemScanTime = Date.now();
-                this.lastItemScanResult = res;
+                this.dataCount.set(res);
-                return cb(null, res);
+                return cb(null, this.dataCount.results());
             });
         });
         return undefined;
     }
 
-    getDataManaged(c, query, log, cb) {
+    _handleResults(res, isVersioned) {
-        const dataManaged = {
+        const total = { curr: 0, prev: 0 };
-            total: 0,
+        const locations = {};
-            locations: {},
-        };
-        c.find(query).toArray((err, entries) => {
-            if (err) {
-                log.error('error occured in mongo client', {
-                    method: 'getDataManaged',
-                    error: err,
-                });
-                return cb(err);
-            }
-            entries.forEach(entry => {
-                const { backends } = entry.value.replicationInfo;
-                const objMD = entry.value;
-                dataManaged.total += objMD['content-length'];
-                if (!dataManaged.locations[objMD.dataStoreName]) {
-                    dataManaged.locations[objMD.dataStoreName] =
-                        objMD['content-length'];
-                } else {
-                    dataManaged.locations[objMD.dataStoreName] +=
-                        objMD['content-length'];
-                }
-                // replication list
-                backends.forEach(location => {
-                    const { site } = location;
-                    if (!dataManaged.locations[site]) {
-                        dataManaged.locations[site] = objMD['content-length'];
-                    } else {
-                        dataManaged.locations[site] += objMD['content-length'];
-                    }
-                });
-            });
-            return cb(null, {
-                objCount: entries.length,
-                dataManaged,
-            });
-        });
-    }
 
-    getDataInfoNoVer(c, log, callback) {
+        Object.keys(res.nullData).forEach(loc => {
-        return this.getDataManaged(c, {}, log, (err, res) => {
+            const bytes = res.nullData[loc];
-            if (err) {
+            const locName = loc === 'mem' || loc === 'file' ?
-                log.error('error occured in mongo client', {
+                'us-east-1' : loc;
-                    method: 'getDataInfoNoVer',
+            if (!locations[locName]) {
-                    error: err,
+                locations[locName] = { curr: 0, prev: 0 };
-                });
-                return callback(err);
             }
-            const dataManaged = {
+            total.curr += bytes;
-                total: { curr: 0, prev: 0 },
+            locations[locName].curr += bytes;
-                locations: {},
+        });
-            };
+        if (isVersioned) {
-            const data = res.dataManaged.locations;
+            Object.keys(res.versionData).forEach(loc => {
-            // add total
+                const bytes = res.versionData[loc];
-            dataManaged.total.curr += res.dataManaged.total;
+                const locName = loc === 'mem' || loc === 'file' ?
-            Object.keys(data).forEach(loc => {
+                    'us-east-1' : loc;
-                if (!dataManaged.locations[loc]) {
+                if (!locations[locName]) {
-                    dataManaged.locations[loc] = { prev: 0, curr: 0 };
+                    locations[locName] = { curr: 0, prev: 0 };
                 }
-                dataManaged.locations[loc].curr = data[loc];
+                total.prev += bytes;
-            });
+                locations[locName].prev += bytes;
-            return callback(null, {
-                objects: res.objCount,
-                versions: 0,
-                dataManaged,
-            });
             });
         }
-
+        Object.keys(res.masterData).forEach(loc => {
-    getDataInfoVer(c, mstVerIds, log, callback) {
+            const bytes = res.masterData[loc];
-        // query for removing placeholder entries
+            const locName = loc === 'mem' || loc === 'file' ?
-        /* eslint-disable quote-props */
+                'us-east-1' : loc;
-        const queryFilter = {
+            if (!locations[locName]) {
-            '$or': [
+                locations[locName] = { curr: 0, prev: 0 };
-                { '_id': { '$regex': /\0.*$/g } },
+            }
-                { 'value.isNull': { '$exists': true } },
+            total.curr += bytes;
-                {
+            locations[locName].curr += bytes;
-                    'value.nullVersionId': { '$exists': false },
+            if (isVersioned) {
-                    'value.versionId': { '$exists': false },
+                total.prev -= bytes;
+                locations[locName].prev -= bytes;
+            }
+        });
+        return {
+            versions: isVersioned ?
+                res.versionCount - res.masterCount : 0,
+            objects: res.masterCount + res.nullCount,
+            dataManaged: {
+                total,
+                locations,
             },
-            ],
         };
-        /* eslint-enable quote-props */
-        return async.series({
-            master: done => {
-                // query for getting only master version entries
-                const mstFilter = [];
-                /* eslint-disable quote-props */
-                mstVerIds.forEach(id => {
-                    if (id) {
-                        mstFilter.push({ 'value.versionId': { '$eq': id } });
-                    }
-                });
-                const query = {
-                    '$and': [
-                        queryFilter,
-                        { '$or': (mstFilter.length ? mstFilter : [{}]) },
-                    ],
-                };
-                /* eslint-enable quote-props */
-                return this.getDataManaged(c, query, log, done);
-            },
-            archived: done => {
-                // query for getting only versioned entries
-                const mstFilter = [];
-                /* eslint-disable quote-props */
-                mstVerIds.forEach(id => {
-                    if (id) {
-                        mstFilter.push({ 'value.versionId': { '$ne': id } });
-                    }
-                });
-                const query = {
-                    '$and': [
-                        queryFilter,
-                        { '$and': (mstFilter.length ? mstFilter : [{}]) },
-                    ],
-                };
-                /* eslint-enable quote-props */
-                return this.getDataManaged(c, query, log, done);
-            },
-        }, (err, res) => {
-            if (err) {
-                log.error('error occured in mongo client', {
-                    method: 'getDataInfoVer',
-                    error: err,
-                });
-                return callback(err);
-            }
-            const dataManaged = {
-                total: { curr: 0, prev: 0 },
-                locations: {},
-            };
-            const mstData = res.master.dataManaged.locations;
-            const verData = res.archived.dataManaged.locations;
-
-            // add total
-            dataManaged.total.curr += res.master.dataManaged.total;
-            dataManaged.total.prev += res.archived.dataManaged.total;
-
-            Object.keys(mstData).forEach(loc => {
-                if (!dataManaged.locations[loc]) {
-                    dataManaged.locations[loc] = { prev: 0, curr: 0 };
-                }
-                dataManaged.locations[loc].curr = mstData[loc];
-            });
-            Object.keys(verData).forEach(loc => {
-                if (!dataManaged.locations[loc]) {
-                    dataManaged.locations[loc] = { prev: 0, curr: 0 };
-                }
-                dataManaged.locations[loc].prev = verData[loc];
-            });
-
-            return callback(null, {
-                objects: res.master.objCount,
-                versions: res.archived.objCount,
-                dataManaged,
-            });
-        });
     }
 
     getObjectMDStats(bucketName, bucketInfo, log, callback) {
@@ -1154,60 +1168,109 @@ class MongoClientInterface {
             name: bucketName,
             location: bucketInfo.getLocationConstraint(),
         };
-        if (bucketInfo && bucketInfo._versioningConfiguration &&
+
-            (bucketInfo._versioningConfiguration.Status === 'Suspended' ||
+        const mstFilter = {
-             bucketInfo._versioningConfiguration.Status === 'Enabled')) {
+            '_id': { $regex: /^[^\0]+$/ },
-            // if versioning is enabled
+            'value.versionId': { $exists: true },
-            c.distinct('_id').then(keys => {
+        };
-                const trimmedKeys = keys.map(key => key.replace(/\0.*$/g, ''));
+        const verFilter = { _id: { $regex: /\0/ } };
-                const uniqKeys = trimmedKeys.filter(
+        const nullFilter = {
-                    (key, index, self) => self.indexOf(key) === index);
+            '_id': { $regex: /^[^\0]+$/ },
-                // for each uniqKey get master version id
+            'value.versionId': { $exists: false },
-                return async.map(uniqKeys, (key, done) => {
+        };
-                    this.getLatestVersion(c, key, log, (err, mst) => {
+        const grpObj = {
+            _id: '$value.dataStoreName',
+            bytes: { $sum: '$value.content-length' },
+        };
+        const grpRepObj = {
+            _id: '$value.replicationInfo.backends.site',
+            bytes: { $sum: '$value.content-length' },
+        };
+
+        const _handleCount = (err, entries, cb) => {
             if (err) {
-                            if (err.NoSuchKey) {
+                return cb(err);
-                                log.debug('NoSuchKey master info', {
+            }
-                                    method: 'getObjectMDStats',
+            return cb(null,
+                entries && entries.length > 0 ? entries.length : 0);
+        };
+        const _handleEntries = (err, entries, cb) => {
+            if (err) {
+                return cb(err);
+            }
+            const results = {};
+            if (entries) {
+                entries.forEach(entry => {
+                    results[entry._id] = entry.bytes;
+                });
+            }
+            return cb(null, results);
+        };
+
+        const _handleMongo = (cursor, cb) => async.parallel({
+            count: done => {
+                const tmpCursor = cursor.clone();
+                tmpCursor.toArray(
+                    (err, res) => _handleCount(err, res, done));
+            },
+            data: done => {
+                const tmpCursor = cursor.clone();
+                tmpCursor.group(grpObj)
+                .toArray((err, res) => _handleEntries(err, res, done));
+            },
+            repData: done => {
+                const tmpCursor = cursor.clone();
+                tmpCursor.unwind('$value.replicationInfo.backends')
+                .group(grpRepObj)
+                .toArray((err, res) => _handleEntries(err, res, done));
+            },
+        }, (err, res) => {
+            if (err) {
+                log.error('Error when processing mongo entries', {
+                    method: '_handleMongo',
                     error: err,
                 });
-                                return done();
+                return cb(err);
             }
-                            log.error('unable to retrieve master info', {
+            const retResult = {
-                                method: 'getObjectMDStats',
+                count: res.count,
-                                error: err,
+                data: Object.assign({}, res.data),
-                            });
+            };
-                            return done(err);
+            Object.keys(res.repData).forEach(site => {
+                if (!retResult.data[site]) {
+                    retResult.data[site] = 0;
                 }
-                        return done(null, mst.versionId);
+                retResult.data[site] += res.repData[site];
             });
-                }, (err, mstVerIds) => {
+            return cb(null, retResult);
+        });
+
+        const mstCursor = c.aggregate().match(mstFilter);
+        const verCursor = c.aggregate().match(verFilter);
+        const nullCursor = c.aggregate().match(nullFilter);
+        async.parallel({
+            version: done => _handleMongo(verCursor, done),
+            null: done => _handleMongo(nullCursor, done),
+            master: done => _handleMongo(mstCursor, done),
+        }, (err, res) => {
             if (err) {
                 return callback(err);
             }
-                    return this.getDataInfoVer(c, mstVerIds, log,
+            const resObj = {
-                    (err, res) => {
+                masterCount: res.master.count || 0,
-                        if (err) {
+                masterData: res.master.data || {},
-                            return callback(err);
+                nullCount: res.null.count || 0,
-                        }
+                nullData: res.null.data || {},
-                        return callback(null, {
+                versionCount: res.version.count || 0,
-                            bucketList: retBucketInfo,
+                versionData: res.version.data || {},
-                            counter: res,
+            };
+            const bucketStatus = bucketInfo.getVersioningConfiguration();
+            const isVer = (bucketStatus && (bucketStatus.Status === 'Enabled' ||
+                bucketStatus.Status === 'Suspended'));
+            const retResult = this._handleResults(resObj, isVer);
+            retResult.bucketList = retBucketInfo;
+            return callback(null, retResult);
         });
-                    });
-                });
-            }).catch(callback);
-        } else {
-            this.getDataInfoNoVer(c, log, (err, res) => {
-                if (err) {
-                    return callback(err);
-                }
-                return callback(null, {
-                    bucketList: retBucketInfo,
-                    counter: res,
-                });
-            });
-        }
     }
 }
 

package.json

@@ -3,7 +3,7 @@
   "engines": {
     "node": ">=6.9.5"
   },
-  "version": "8.0.0",
+  "version": "8.0.0-test-1.0",
   "description": "Common utilities for the S3 project components",
   "main": "index.js",
   "repository": {

tests/unit/storage/metadata/mongoclient/DataCounter.js

@@ -0,0 +1,363 @@
+const assert = require('assert');
+
+const {
+    NEW_OBJ,
+    NEW_VER,
+    UPDATE_VER,
+    UPDATE_MST,
+    RESTORE,
+    DEL_VER,
+    DEL_MST,
+    DataCounter,
+} = require('../../../../../lib/storage/metadata/mongoclient/DataCounter');
+
+const refZeroObj = {
+    objects: 0,
+    versions: 0,
+    buckets: 0,
+    bucketList: [],
+    dataManaged: {
+        total: { curr: 0, prev: 0 },
+        byLocation: {},
+    },
+};
+
+const refSingleObj = {
+    objects: 2,
+    versions: 0,
+    buckets: 0,
+    bucketList: [],
+    dataManaged: {
+        total: { curr: 200, prev: 0 },
+        byLocation: {
+            locationOne: { curr: 200, prev: 0 },
+        },
+    },
+};
+
+const refSingleObjVer = {
+    objects: 1,
+    versions: 1,
+    buckets: 0,
+    bucketList: [],
+    dataManaged: {
+        total: { curr: 100, prev: 100 },
+        byLocation: {
+            locationOne: { curr: 100, prev: 100 },
+        },
+    },
+};
+
+const refMultiObjVer = {
+    objects: 1,
+    versions: 1,
+    buckets: 0,
+    bucketList: [],
+    dataManaged: {
+        total: { curr: 200, prev: 200 },
+        byLocation: {
+            locationOne: { curr: 100, prev: 100 },
+            locationTwo: { curr: 100, prev: 100 },
+        },
+    },
+};
+
+const refMultiObj = {
+    objects: 2,
+    versions: 0,
+    buckets: 0,
+    bucketList: [],
+    dataManaged: {
+        total: { curr: 400, prev: 0 },
+        byLocation: {
+            locationOne: { curr: 200, prev: 0 },
+            locationTwo: { curr: 200, prev: 0 },
+        },
+    },
+};
+
+// eslint-disable-next-line quote-props
+const singleSite = size => ({
+    'content-length': size,
+    dataStoreName: 'locationOne',
+    replicationInfo: {
+        backends: [],
+    },
+});
+
+// eslint-disable-next-line quote-props
+const multiSite = (size, isComplete) => ({
+    'content-length': size,
+    dataStoreName: 'locationOne',
+    replicationInfo: {
+        backends: [{
+            site: 'locationTwo',
+            status: isComplete ? 'COMPLETED' : 'PENDING',
+        }],
+    },
+});
+
+const dataCounter = new DataCounter();
+
+describe('DataCounter Class', () => {
+    it('should create a zero object', () => {
+        dataCounter.set(refZeroObj);
+        assert.deepStrictEqual(dataCounter.results(), refZeroObj);
+    });
+
+    it('should skip dataCounter methods if initial values are not set', () => {
+        const testCounter = new DataCounter();
+        testCounter.addObject(singleSite(100), null, NEW_OBJ);
+        assert.deepStrictEqual(testCounter.results(), refZeroObj);
+    });
+});
+
+describe('DataCounter::addObject', () => {
+    const tests = [
+        {
+            it: 'should correctly update DataCounter, new object one site',
+            init: refZeroObj,
+            input: [singleSite(100), null, NEW_OBJ],
+            expectedRes: {
+                objects: 1, versions: 0,
+                dataManaged: {
+                    total: { curr: 100, prev: 0 },
+                    byLocation: {
+                        locationOne: { curr: 100, prev: 0 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, new object multi site',
+            init: refZeroObj,
+            input: [multiSite(100, true), null, NEW_OBJ],
+            expectedRes: {
+                objects: 1, versions: 0,
+                dataManaged: {
+                    total: { curr: 200, prev: 0 },
+                    byLocation: {
+                        locationOne: { curr: 100, prev: 0 },
+                        locationTwo: { curr: 100, prev: 0 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, overwrite single site',
+            init: refSingleObj,
+            input: [singleSite(100), singleSite(50), NEW_OBJ],
+            expectedRes: {
+                objects: 2, versions: 0,
+                dataManaged: {
+                    total: { curr: 250, prev: 0 },
+                    byLocation: {
+                        locationOne: { curr: 250, prev: 0 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, overwrite multi site',
+            init: refMultiObj,
+            input: [multiSite(100, true), multiSite(50, true), NEW_OBJ],
+            expectedRes: {
+                objects: 2, versions: 0,
+                dataManaged: {
+                    total: { curr: 500, prev: 0 },
+                    byLocation: {
+                        locationOne: { curr: 250, prev: 0 },
+                        locationTwo: { curr: 250, prev: 0 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, new version single site',
+            init: refSingleObj,
+            input: [singleSite(100), singleSite(50), NEW_VER],
+            expectedRes: {
+                objects: 2, versions: 1,
+                dataManaged: {
+                    total: { curr: 250, prev: 50 },
+                    byLocation: {
+                        locationOne: { curr: 250, prev: 50 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, new version multi site',
+            init: refMultiObj,
+            input: [multiSite(100, true), multiSite(50, true), NEW_VER],
+            expectedRes: {
+                objects: 2, versions: 1,
+                dataManaged: {
+                    total: { curr: 500, prev: 100 },
+                    byLocation: {
+                        locationOne: { curr: 250, prev: 50 },
+                        locationTwo: { curr: 250, prev: 50 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly ignore pending status, multi site',
+            init: refZeroObj,
+            input: [multiSite(100, false), null, NEW_OBJ],
+            expectedRes: {
+                objects: 1, versions: 0,
+                dataManaged: {
+                    total: { curr: 100, prev: 0 },
+                    byLocation: {
+                        locationOne: { curr: 100, prev: 0 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, ' +
+            'replication completion update in master object',
+            init: refSingleObj,
+            input: [multiSite(100, true), multiSite(100, false), UPDATE_MST],
+            expectedRes: {
+                objects: 2, versions: 0,
+                dataManaged: {
+                    total: { curr: 300, prev: 0 },
+                    byLocation: {
+                        locationOne: { curr: 200, prev: 0 },
+                        locationTwo: { curr: 100, prev: 0 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, ' +
+            'replication completion update in versioned object',
+            init: refSingleObjVer,
+            input: [multiSite(100, true), multiSite(100, false), UPDATE_VER],
+            expectedRes: {
+                objects: 1, versions: 1,
+                dataManaged: {
+                    total: { curr: 100, prev: 200 },
+                    byLocation: {
+                        locationOne: { curr: 100, prev: 100 },
+                        locationTwo: { curr: 0, prev: 100 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, ' +
+            'restoring versioned object as master',
+            init: refMultiObjVer,
+            input: [multiSite(100, true), multiSite(100, true), RESTORE],
+            expectedRes: {
+                objects: 2, versions: 0,
+                dataManaged: {
+                    total: { curr: 400, prev: 0 },
+                    byLocation: {
+                        locationOne: { curr: 200, prev: 0 },
+                        locationTwo: { curr: 200, prev: 0 },
+                    },
+                },
+            },
+        },
+    ];
+    tests.forEach(test => it(test.it, () => {
+        const { expectedRes, input, init } = test;
+        dataCounter.set(init);
+        dataCounter.addObject(...input);
+        const testResults = dataCounter.results();
+        Object.keys(expectedRes).forEach(key => {
+            if (typeof expectedRes[key] === 'object') {
+                assert.deepStrictEqual(testResults[key], expectedRes[key]);
+            } else {
+                assert.strictEqual(testResults[key], expectedRes[key]);
+            }
+        });
+    }));
+});
+
+describe('DataCounter::delObject', () => {
+    const tests = [
+        {
+            it: 'should correctly update DataCounter, ' +
+            'delete master object single site',
+            init: refMultiObj,
+            input: [singleSite(100), DEL_MST],
+            expectedRes: {
+                objects: 1, versions: 0,
+                dataManaged: {
+                    total: { curr: 300, prev: 0 },
+                    byLocation: {
+                        locationOne: { curr: 100, prev: 0 },
+                        locationTwo: { curr: 200, prev: 0 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, ' +
+            'delete master object multi site',
+            init: refMultiObj,
+            input: [multiSite(100, true), DEL_MST],
+            expectedRes: {
+                objects: 1, versions: 0,
+                dataManaged: {
+                    total: { curr: 200, prev: 0 },
+                    byLocation: {
+                        locationOne: { curr: 100, prev: 0 },
+                        locationTwo: { curr: 100, prev: 0 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, ' +
+            'delete versioned object single site',
+            init: refMultiObjVer,
+            input: [singleSite(100), DEL_VER],
+            expectedRes: {
+                objects: 1, versions: 0,
+                dataManaged: {
+                    total: { curr: 200, prev: 100 },
+                    byLocation: {
+                        locationOne: { curr: 100, prev: 0 },
+                        locationTwo: { curr: 100, prev: 100 },
+                    },
+                },
+            },
+        },
+        {
+            it: 'should correctly update DataCounter, ' +
+            'delete versioned object multi site',
+            init: refMultiObjVer,
+            input: [multiSite(100, true), DEL_VER],
+            expectedRes: {
+                objects: 1, versions: 0,
+                dataManaged: {
+                    total: { curr: 200, prev: 0 },
+                    byLocation: {
+                        locationOne: { curr: 100, prev: 0 },
+                        locationTwo: { curr: 100, prev: 0 },
+                    },
+                },
+            },
+        },
+    ];
+
+    tests.forEach(test => it(test.it, () => {
+        const { expectedRes, input, init } = test;
+        dataCounter.set(init);
+        dataCounter.delObject(...input);
+        const testResults = dataCounter.results();
+        Object.keys(expectedRes).forEach(key => {
+            if (typeof expectedRes[key] === 'object') {
+                assert.deepStrictEqual(testResults[key], expectedRes[key]);
+            } else {
+                assert.strictEqual(testResults[key], expectedRes[key]);
+            }
+        });
+    }));
+});