Why is that? We follow the same pattern in several other places in the same
function.
On Aug 2, 2013 12:44 PM, "Justin Gallardo" <notifications@github.com> wrote:

> In lib/util/knife.js:
>
> > @@ -186,7 +186,18 @@ function queryAndRun(baton, args, query, cmd, msg, options, callback) {
> >        length: results.rows.length
> >      });
> >
> > -    async.map(results.rows, execute, function(_, errors) {
> > +    var eachFunc = async.forEach;
> > +    if (options && options.parallelLimit) {
> > +      eachFunc = function(arr, iterator, callback) {
> > +        var parallelLimit = options.parallelLimit;
> > +        if (typeof parallelLimit === 'function') {
>
> I think that parallelLimit should be specified as only a number, or only
> a function. In this case, I would rename it to parallelLimitFunc and
> assume it will always be a function.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/racker/dreadnot/pull/54/files#r5561407>
> .
>