Does `findRandom` return an array of sequential documents? about mongoose-simple-random HOT 4 CLOSED

OK I've started looking into this, and although we get a significant slowdown, I believe that having more random docs will be better. Will update in a few days once I've got the algo down.

from mongoose-simple-random.

I think that's how it currently works, and I agree that it's probably non-optimal as far as functionality goes. I'll happily look at any submitted PR.

from mongoose-simple-random.

No solution here I'm afraid. The approach I landed on was to choose one random document n times, using your skip approach, and then handle the risk of the same document being chosen twice. That's n queries!

From the research I did: an option some people used was to include a random value in a field of every document in the DB, and then use that field for ordering. But it seems to me that those values should be updated each time (at least for the retrieved documents), otherwise later queries may see similar repeated blocks of documents.

I wonder if a mongo aggregate query could somehow hash (the objid of each document combined with a fresh random value supplied each time). In theory that could provide a random an unpredictable value we could sort on. It might be pretty heavy on large collections though.

from mongoose-simple-random.

I don't use this plugin due to sequential issue.

I have quick coded this and it works like wanted.
It have 0 chance to take same documents twice (cf _.sample(_.range(count), nb);)

As I do use external dependancy as async and underscore, i can't make PR with this code.

var async = require("async");
var _ = require("underscore");

module.exports = exports = function(schema) {
    return schema.statics.pickRandom = function(query, nb, next) {
        return this.count(query, (err, count) => {
            if (err) {
                return next(err);
            }
            var randomSkip = _.sample(_.range(count), nb);
            return async.map(randomSkip, (skip, next) => {
                return this.findOne(query).skip(skip).exec(next);
            }, function(err, memories) {
                return next(err, memories, count);
            });
        });
    };
};

from mongoose-simple-random.