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

After a few hours the stream read() loop gets stuck, this fixed it for me. #135

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@unthingable

unthingable commented Nov 27, 2011

No description provided.

@hamtie

This comment has been minimized.

Show comment
Hide comment
@hamtie

hamtie Dec 15, 2011

My twitter stream frequently hangs. This seems like it might fix the issue. Have you had any more hanging since you made this patch? I would love to see this fix in the main repo.

hamtie commented Dec 15, 2011

My twitter stream frequently hangs. This seems like it might fix the issue. Have you had any more hanging since you made this patch? I would love to see this fix in the main repo.

@unthingable

This comment has been minimized.

Show comment
Hide comment
@unthingable

unthingable Dec 19, 2011

Thanks. Nope, this seems to have fixed the hanging for me and I have heard one other similar report.

unthingable commented Dec 19, 2011

Thanks. Nope, this seems to have fixed the hanging for me and I have heard one other similar report.

@bcambel

This comment has been minimized.

Show comment
Hide comment
@bcambel

bcambel Dec 29, 2011

Hanging problem also exists in my setup.. Well I'm gonna give a try to this small fix.

bcambel commented Dec 29, 2011

Hanging problem also exists in my setup.. Well I'm gonna give a try to this small fix.

@bcambel

This comment has been minimized.

Show comment
Hide comment
@bcambel

bcambel Dec 29, 2011

I can confirm that my app is working for the last 16h+ with this fix processing around 100 tweets per 3 seconds

bcambel commented Dec 29, 2011

I can confirm that my app is working for the last 16h+ with this fix processing around 100 tweets per 3 seconds

@hamtie

This comment has been minimized.

Show comment
Hide comment
@hamtie

hamtie Feb 16, 2012

Just wanted to note that I've pulled this into my fork and my twitter agent has been running for 2 days now.

hamtie commented Feb 16, 2012

Just wanted to note that I've pulled this into my fork and my twitter agent has been running for 2 days now.

@unthingable

This comment has been minimized.

Show comment
Hide comment
@unthingable

unthingable Mar 4, 2012

Thanks to everyone who pulled and tested this.

unthingable commented Mar 4, 2012

Thanks to everyone who pulled and tested this.

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