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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

shoryuken not polling from SQS #415

Closed
ACATTAN opened this issue Jul 24, 2017 · 3 comments
Closed

shoryuken not polling from SQS #415

ACATTAN opened this issue Jul 24, 2017 · 3 comments

Comments

@ACATTAN
Copy link

ACATTAN commented Jul 24, 2017

Hey guys,

I just tried out version 3.1.5. When launching shoryuken it reads SQS messages in the queue just fine. However, subsequent messages arriving to the queue are not read any more.
The delay set in my yml config file is 30 (sec).
Trying with version 2.1.3 - works well (same configuration exactly). Also worked well in version 1.x.
Any idea?

Thanks!
Ariel
phstc pushed a commit that referenced this issue Jul 24, 2017
It seems like it must call dispatch_loop in the ensure block
d10d16f 
Otherwise, it gets lost with long delays.

Fixes #415
@phstc phstc mentioned this issue Jul 24, 2017
Merged
phstc added a commit that referenced this issue Jul 24, 2017
@phstc
Merge pull request #416 from phstc/hotfix/415-ensure-to-call-dispatch…
c64c0f8 
…_loop

It seems like it must call dispatch_loop in the ensure block

Fixes #415
@phstc
Copy link
Collaborator

phstc commented Jul 24, 2017

@ACATTAN thanks for reporting this! Fixed in 3.1.6. Could you give it another try?

@ACATTAN
Copy link
Author

ACATTAN commented Jul 24, 2017

Yes of course. I'll test it tomorrow and update.

@ACATTAN
Copy link
Author

ACATTAN commented Jul 25, 2017

Just checked it and it seems to work fine in 3.1.6, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@phstc @ACATTAN