Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable search indexes by default #412

Closed
behrad opened this issue Sep 15, 2014 · 9 comments
Closed

Disable search indexes by default #412

behrad opened this issue Sep 15, 2014 · 9 comments

Comments

@behrad
Copy link
Collaborator

behrad commented Sep 15, 2014

  1. They are leaky on job removal
  2. They waste memory when not needed
  3. They create their own (reds module) redis connections, which is not controlled by kue. That currently causes node.js process not to exit after shutdown
@Suor
Copy link

Suor commented Oct 19, 2014

+1

1 similar comment
@maxvgi
Copy link

maxvgi commented Oct 20, 2014

+1

@bradvogel
Copy link

The readme doesn't really explain what the search indexes are for. What's the tradeoff of disabling them? Will the UI still work?

@behrad
Copy link
Collaborator Author

behrad commented Mar 25, 2015

search indexes mainly was built for UI,
tradeoff of enabling them is higher memory consumption and sometimes leaking indexes, and more (unmanaged by Kue) connections to redis.

Will the UI still work?

unfortunately NO! :(

@bradvogel
Copy link

OK thanks. But I'm noticing the UI still updates with search indexes turned off. Can you describe what parts of the UI won't work with this turned off?

@behrad
Copy link
Collaborator Author

behrad commented Mar 25, 2015

which version are you on? how are you bootstrapping your ui ?

@behrad
Copy link
Collaborator Author

behrad commented Mar 25, 2015

Can you describe what parts of the UI won't work with this turned off?

the search text field

@bradvogel
Copy link

0.8.12. But that makes sense that just the search text field doesn't work. The rest of the UI seems to function fine.

@behrad
Copy link
Collaborator Author

behrad commented Mar 25, 2015

this is merged with latest (0.9.0) not the 0.8.12 so in your Kue also the search text field should work as before :)

The rest of the UI seems to function fine.

sure, it sould

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

No branches or pull requests

4 participants