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

queue behavior #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

queue behavior #8

wants to merge 2 commits into from

Conversation

chebyte
Copy link

@chebyte chebyte commented Mar 6, 2015

@soveran thanks for this gem, is very useful for us!, I just added a fews methods for queue behavior without looping, is you think this is necessary, please add this changes

thanks!

@soveran
Copy link
Owner

soveran commented Mar 6, 2015

Hey @chebyte, thanks for the patch and for the kind words! Can you tell me more about the use case for this change? The reason why I ask is because BRPOPLPUSH provides some guarantees so that if something goes wrong with the worker, the item is kept in a backup queue. Tell me more about how you plan to use it.

@chebyte
Copy link
Author

chebyte commented Mar 6, 2015

We use this methods so we can use OST like a queue instead a loop. We could have use only Redis, but we are already are using OST for several reasons, so we went that way. We aren't sure that this changes mean a great change on the gem, but in the spirit of open source, we would like to share. Please feel free to close this PR if you feel like.

thanks!

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

Successfully merging this pull request may close these issues.

None yet

2 participants