Mailing-List: contact kafka-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: kafka-dev@incubator.apache.org
Date: Thu, 1 Nov 2012 09:43:12 +0000 (UTC)
From: "Yang Ye (JIRA)" <jira@apache.org>
To: kafka-dev@incubator.apache.org
Message-ID: <2040883530.55296.1351762992255.JavaMail.jiratomcat@arcas>
In-Reply-To: <726507185.9758.1346202187638.JavaMail.jiratomcat@arcas>
Subject: [jira] [Updated] (KAFKA-491) KafkaRequestHandler needs to handle
 exceptions
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/KAFKA-491?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Yang Ye updated KAFKA-491:
--------------------------

    Attachment: kafka-491-v2.diff


Fixed comments 1.1, 1.2, 1.4, 2

For 1.3,
It seems not easy to do that, if we have some common variable outside, then the type of this variable has only to be vague, then when we log it, we cannot get the detailed info inside it. 
                
> KafkaRequestHandler needs to handle exceptions
> ----------------------------------------------
>
>                 Key: KAFKA-491
>                 URL: https://issues.apache.org/jira/browse/KAFKA-491
>             Project: Kafka
>          Issue Type: Bug
>          Components: core
>            Reporter: Jun Rao
>              Labels: bugs
>             Fix For: 0.8
>
>         Attachments: kafka-491-v1.diff, kafka-491-v2.diff
>
>
> Currently, if apis.handle() throws an exception (e.g., if the broker receives an invalid request), KafkaRequestHandler will die. We need to handle exceptions properly.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira