Yes, this will take care of it, this._writeOnlyInstances.concat(this._instance)



On Wed, Feb 5, 2014 at 4:27 PM, Xiao Ma <notifications@github.com> wrote:

> In lib/MultiWriteCacheGroup.js:
>
> > @@ -128,4 +129,9 @@ MultiWriteCacheGroup.prototype._applyWrites = function (op, arguments) {
> >    return Q.all(promises)
> >  }
> >
> > +/** @override */
> > +MultiWriteCacheGroup.prototype.getPendingRequestsCount = function () {
> > +  return cacheUtils.mergePendingRequestCounts(this._writeOnlyInstances.concat(this._instance))
>
> You want to add the count from the primary instance as well.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/Obvious/zcache/pull/35/files#r9485740>
> .
>