-
Notifications
You must be signed in to change notification settings - Fork 222
2.0 dev (WIP) #112
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
2.0 dev (WIP) #112
Conversation
|
🔥 |
|
🙏 🎉 |
|
@LewisJEllis thank you! I didn't know there was a 2.0 in the works! I'm very happy to help with some of the development here. So far bee-queue has been working really well for us and I want to see this project continue to develop. |
|
Would you consider a PR for transferring the current 2.0 codebase to TypeScript? |
|
I'm sure you've been busy on other stuff, @LewisJEllis, but if you've got a sec could you update what's done, not done, and what others could help on? |
|
any updates? |
|
What's the status of this PR and @LewisJEllis involvement with bee-queue? |
|
Hey all :) I am the author of ConveyorMQ which I have built with the intention of it being a fast, robust and reliable distributed task queue for Node using Redis. My intention is to make ConveyorMQ the most performant, feature-full and defacto-standard job/task queue system for Node. ConveyorMQ already implements all of bee-queue's features, as well as the following:
Additionally, my road map currently includes:
I would be grateful for any feedback you might have to offer :) |
|
@jasrusable awesome project! Just curious - have you considered using Redis Streams (introduced in Redis 5.0)? If we were to build bee-queue again from the ground up, we likely would have used Streams since they simplify so much (see this comment). |
|
This PR has many conflicts and appears abandoned, closing. |
I have moved everything over to ioredis, torn out all the callback stuff, and gotten all the tests back to passing! It took a while, but it's pretty much the main hurdle/technical risk for making the 2.0 plan work. That's enough to warrant a WIP branch. I will ping people here when it's less WIP and more ready for review.
Some notes/thoughts:
defineCommandcapability for lua scripts - could obviate a lot of the load-script-cache stuff we've implementedif (cb) helpers.asCallback(promise, cb);.processhandler should just be anasyncfunc is greatpromise-callbacksdown to a devDependencywaitOnandwithTimeoutthough, so plucked those two into helpers fileOther todos:
Refs #104