Add hook.datastore.[lpush, lpushx, rpush, rpushx]

[christiangenco]

hook.datastore.recent is timing out when I call it (I think "COUNT", 100 might be waiting for 100 matches to return?), but I'd still like a way to keep track of all of my keys.

My plan is to store them in a "keys" array. I could do this:

var key="foo", value="bar";
hook.datastore.get("keys", function(err, keys){
  keys = keys || [];
  keys.push(key);
  hook.datastore.set("keys", keys, function(err, result){
    hook.datastore.set(key, value, function(err, result){
      hook.res.end(result);
    })
  })
});

but it'd be easier/faster/more efficient for me to do this:

var key="foo", value="bar";
hook.datastore.lpush("keys", key, function(err, result){
  hook.datastore.set(key, value, function(err, result){
    hook.res.end(result);
  });
});

[Marak]

@christiangenco -

Thanks for the report. We are now looking to improve the datastore API.

First, recent should be working again. I had hot-patched this on the server a few weeks and just now backported the fix with 248c4ea

I'm thinking of just implementing datastore.all which would use SCAN to return all records which matched the users root name.

I'd like to try to keep the datastore api simple, but if we are going to hit performance issues, we should start thinking about optimizing for redis.

Let me know. A pull request would help.

[Marak]

CC @zanona

This might also solve the issue you are looking to address.