Skip to content

Instantly share code, notes, and snippets.

@cadorn
Created February 2, 2012 19:39
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save cadorn/1725325 to your computer and use it in GitHub Desktop.
Save cadorn/1725325 to your computer and use it in GitHub Desktop.
Q.Throttle()
/**
* Copyright: 2012 Christoph Dorn <christoph@christophdorn.com>
* License: MIT
* Source: https://gist.github.com/1725325
*
* NodeJS Example:
*
* // Requirements: `npm install q`
*
* var Q = require("q");
*
* // Require this module (to add `Q.Throttle` to the `Q` API)
* Q.Throttle = require("q-throttle").Throttle;
*
* // Maximum of 3 unresolved promises at a time
* var throttle = Q.Throttle(3);
*
* for (var i=0 ; i < 10 ; i++) {
* throttle.when([i], function(i) {
* // Never more than 3 unresolved doDeferredWork() promises
* return doDeferredWork(i).then(function() {
* });
* });
* }
*
* throttle.on("done", function()
* {
* });
*
*/
var Q = require("q"),
EVENTS = require("events");
var Throttle = exports.Throttle = function(max)
{
if (!(this instanceof Throttle))
return new Throttle(max);
this.count = 0;
this.buffer = [];
this.max = max;
};
Throttle.prototype = new EVENTS.EventEmitter();
Throttle.prototype.when = function(args, func)
{
var self = this,
result;
if (self.count >= self.max)
{
self.buffer.push([args, func]);
return;
}
self.count += 1;
result = func.apply(null, args);
if (!Q.isPromise(result))
{
throw new Error("Throttled function call did not return a promise!");
}
Q.when(result, function()
{
self.count -= 1;
if (self.buffer.length > 0)
{
var info = self.buffer.shift();
self.when(info[0], info[1]);
}
else
if (self.count === 0)
{
self.emit("done");
}
});
}
exports.Throttle_Test = function()
{
var deferred = Q.defer();
try
{
// Maximum of 3 unresolved promises at a time
var throttle = new Throttle(3),
list = [],
i,
count = 0;
console.log("Running Throttle_Test:");
for (i=0 ; i < 10 ; i++)
{
list.push(function()
{
return Q.delay(200 + (100 * i%3/10));
});
}
for (i=0 ; i < list.length ; i++)
{
console.log(" triggering " + i);
throttle.when([i], function(i)
{
// Only 3 unresolved promises will run at a time
console.log(" starting " + i);
count += 1;
if (count > 3) throw new Error("More than 3 unresolved promises at the same time!");
return list[i]().then(function()
{
console.log(" " + i + " done");
count -= 1;
});
});
}
console.log(" triggering done");
throttle.on("done", function()
{
console.log("Throttle_Test OK!");
deferred.resolve();
});
}
catch(e)
{
deferred.reject(e);
}
return deferred.promise;
}
@kriskowal
Copy link

Cool.

Might consider a throttle decorator.

var readFile = throttle(FD_MAX, readFile);

I wonder whether there are data-lock risks with this approach; like being unable to make progress because the throttle won’t release a promise. It might not be a good idea to throttle an entire system of promises.

Were you in the discussions about making a promise library inheritable, so you could pool promises for the purpose of counting or checking which promises are outstanding?

@cadorn
Copy link
Author

cadorn commented Feb 2, 2012

@kriskowal I was loosely following that discussion. Don't really remember what the outcome was.

I am primarily using it to throttle system calls in a loop. Looks like I still need to add rejection handeling.

Regarding throttled FS access. See: https://github.com/isaacs/node-graceful-fs

Maybe this can be combined.

@kriskowal
Copy link

@cadorn I recall the graceful FS module. It might be better to create a decorator that employs control theory to zero in on the ideal attempt rate. Perhaps additive-increase–multiplicative-decrease like TCP. But control systems are fickle, especially poorly designed ones, and the trick will be to close the adapter over enough of the system that it receives high-signal (low-noise) information about the availability of the underlying resource. In this case, file descriptors.

@cadorn
Copy link
Author

cadorn commented Feb 2, 2012

@kriskowal Yeah I agree. Should be simple enough within one nodejs instance. Once multiple processes come into play we need to get data from the overall system. That is where PINF is headed. The goal is to make a PINF based system completely Idempotently provisioned and self aware.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment