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

Exception in twitter stream #8

Open
ericnewcomer opened this issue Sep 7, 2016 · 4 comments
Open

Exception in twitter stream #8

ericnewcomer opened this issue Sep 7, 2016 · 4 comments
Assignees

Comments

@ericnewcomer
Copy link
Member

In the mage log, there are instances of the following exception. Need to investigate and resolve.

ERROR [2016-09-07 19:59:53,136] io.rapidpro.mage.twitter.TwitterStream: Exception in Twitter stream
! twitter4j.JSONException: JSONObject["percent_full"] not found.
! at twitter4j.JSONObject.get(JSONObject.java:391) ~[mage.jar:0.1.78]
! at twitter4j.JSONObject.getInt(JSONObject.java:429) ~[mage.jar:0.1.78]
! at com.twitter.hbc.twitter4j.BaseTwitter4jClient.processStallWarning(BaseTwitter4jClient.java:255) ~[mage.jar:0.1.78]
! at com.twitter.hbc.twitter4j.BaseTwitter4jClient.processMessage(BaseTwitter4jClient.java:215) ~[mage.jar:0.1.78]
! at com.twitter.hbc.twitter4j.BaseTwitter4jClient.parseMessage(BaseTwitter4jClient.java:129) ~[mage.jar:0.1.78]
! at com.twitter.hbc.twitter4j.BaseTwitter4jClient$1.run(BaseTwitter4jClient.java:76) ~[mage.jar:0.1.78]
! at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) [na:1.8.0_20]
! at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) [na:1.8.0_20]
@rowanseymour
Copy link
Contributor

This is a bug in the not-so-maintained-any-longer Twitter HBC library twitter/hbc#155

@ericnewcomer
Copy link
Member Author

What's the net effect to us? Does this cause us to miss messages? We should probably fork and fix it as long as we depend on this library.

@rowanseymour
Copy link
Contributor

No effect - happens during creation of a stream to a user with a high number of followers (Twitter sends us FOLLOWS_OVER_LIMIT packet), but stream creation succeeds and we get the DMs.

Agreed we should still fix this - it's been on my todo list forever.

@ericnewcomer
Copy link
Member Author

Ya, I can see why that has lingered then! :neckbeard:

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

No branches or pull requests

2 participants