[ https://issues.apache.org/jira/browse/KAFKA-1140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13830373#comment-13830373 ]
Guozhang Wang commented on KAFKA-1140: -------------------------------------- Created reviewboard https://reviews.apache.org/r/15805/ against branch origin/trunk > Move the decoding logic from ConsumerIterator.makeNext to next > -------------------------------------------------------------- > > Key: KAFKA-1140 > URL: https://issues.apache.org/jira/browse/KAFKA-1140 > Project: Kafka > Issue Type: Bug > Reporter: Guozhang Wang > Assignee: Guozhang Wang > Fix For: 0.8.1 > > Attachments: KAFKA-1140.patch > > > Usually people will write code around consumer like > while(iter.hasNext()) { > try { > msg = iter.next() > // do something > } > catch{ > } > } > ---- > However, the iter.hasNext() call itself can throw exceptions due to decoding > failures. It would be better to move the decoding to the next function call. -- This message was sent by Atlassian JIRA (v6.1#6144)