[ 
https://issues.apache.org/jira/browse/STORM-303?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13990739#comment-13990739
 ] 

ASF GitHub Bot commented on STORM-303:
--------------------------------------

Github user brndnmtthws commented on a diff in the pull request:

    https://github.com/apache/incubator-storm/pull/94#discussion_r12329890
  
    --- Diff: external/storm-kafka/src/jvm/storm/kafka/KafkaUtils.java ---
    @@ -159,12 +171,17 @@ public static ByteBufferMessageSet 
fetchMessages(KafkaConfig config, SimpleConsu
             for (int errors = 0; errors < 2 && msgs == null; errors++) {
                 FetchRequestBuilder builder = new FetchRequestBuilder();
                 FetchRequest fetchRequest = builder.addFetch(topic, 
partitionId, offset, config.fetchSizeBytes).
    -                    clientId(config.clientId).build();
    +                    
clientId(config.clientId).maxWait(config.fetchMaxWait).build();
                 FetchResponse fetchResponse;
                 try {
                     fetchResponse = consumer.fetch(fetchRequest);
                 } catch (Exception e) {
    -                if (e instanceof ConnectException) {
    +                if (e instanceof ConnectException ||
    --- End diff --
    
    Ah, okay.  JDK6 is fine too.  I can repair this.


> Forward port of storm-kafka work
> --------------------------------
>
>                 Key: STORM-303
>                 URL: https://issues.apache.org/jira/browse/STORM-303
>             Project: Apache Storm (Incubating)
>          Issue Type: Bug
>    Affects Versions: 0.9.2-incubating
>         Environment: linux
>            Reporter: Brenden Matthews
>
> This is a placeholder issue for the patch at 
> https://github.com/apache/incubator-storm/pull/94.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to