[jira] [Commented] (KAFKA-1140) Move the decoding logic from ConsumerIterator.makeNext to next

["Guozhang Wang (JIRA)" <ji...@apache.org>]

    [ 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)