Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

/job/search hangs #107

Merged
merged 1 commit into from

4 participants

@grahamscott

The job search call did not invoke query.end() so never returned.

@lynns

I just verified this change fixes the problem. Would love to see it merged so we can query the que.

@tj tj merged commit ecdbedd into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 23, 2012
  1. @grahamscott

    Fixed issue where Reds query did not have end() called on it, causing…

    grahamscott authored
    … /job/search to hang indefinitely
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 lib/http/routes/json.js
View
4 lib/http/routes/json.js
@@ -27,7 +27,7 @@ function getSearch() {
/**
* Get statistics including:
- *
+ *
* - inactive count
* - active count
* - complete count
@@ -176,7 +176,7 @@ exports.updateState = function(req, res){
*/
exports.search = function(req, res){
- getSearch().query(req.query.q, function(err, ids){
+ getSearch().query(req.query.q).end(function(err, ids){
if (err) return res.send({ error: err.message });
res.send(ids);
});
Something went wrong with that request. Please try again.