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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

(DOCS) Update REFERENCE.md to note a performance footgun #2357

Closed
wants to merge 1 commit into from
Closed

(DOCS) Update REFERENCE.md to note a performance footgun #2357

wants to merge 1 commit into from

Commits on May 17, 2022

  1. Update REFERENCE.md to note a performance footgun 

    At my company, we ran `queue.clean(0, "waiting");` on a queue with 1.5 million tasks enqueued (don't ask, cleaning up other peoples' messes is my specialty). This worked locally almost instantly on a couple thousand tasks. But in our lower environments, the Lua script it runs pinned our Redis server to 100% CPU for **over 18 hours** while it worked. The database stopped responding to any commands, and we could not shut it off, even through the GCP hosted databases console.

We had to replace our Redis database (the service stopped responding entirely) for the environment. Luckily, we did not run this on production, or we would have been EXTRA boned. I only blocked the company for around 3 hours before we were able to recover. 馃槩
    @baublet
    baublet committed May 17, 2022
    Configuration menu
    Copy the full SHA
    447d5c1 View commit details
    Browse the repository at this point in the history