You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@streams.apache.org by mf...@apache.org on 2014/05/27 16:01:01 UTC

[5/5] git commit: Merged PR#19 from rdouglas/STREAMS-87

Merged PR#19 from rdouglas/STREAMS-87


Project: http://git-wip-us.apache.org/repos/asf/incubator-streams/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-streams/commit/4580d533
Tree: http://git-wip-us.apache.org/repos/asf/incubator-streams/tree/4580d533
Diff: http://git-wip-us.apache.org/repos/asf/incubator-streams/diff/4580d533

Branch: refs/heads/master
Commit: 4580d533dd94dec08900a17f9499d36467ad77fa
Parents: ec5d348 106eb00
Author: mfranklin <mf...@apache.org>
Authored: Tue May 27 09:49:02 2014 -0400
Committer: mfranklin <mf...@apache.org>
Committed: Tue May 27 09:49:02 2014 -0400

----------------------------------------------------------------------
 .../provider/TwitterTimelineProvider.java       | 23 ++++----------------
 1 file changed, 4 insertions(+), 19 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-streams/blob/4580d533/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/provider/TwitterTimelineProvider.java
----------------------------------------------------------------------
diff --cc streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/provider/TwitterTimelineProvider.java
index b6e4332,49a2d39..72e58d7
--- a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/provider/TwitterTimelineProvider.java
+++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/provider/TwitterTimelineProvider.java
@@@ -137,9 -116,11 +119,12 @@@ public class TwitterTimelineProvider im
              int keepTrying = 0;
  
              // keep trying to load, give it 5 attempts.
-             while (keepTrying < 1)
+             //This value was chosen because it seemed like a reasonable number of times
+             //to retry capturing a timeline given the sorts of errors that could potentially
+             //occur (network timeout/interruption, faulty client, etc.)
+             while (keepTrying < 5)
              {
 +
                  try
                  {
                      statuses = client.getUserTimeline(currentId, paging);