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

Jun Rao commented on KAFKA-649:
-------------------------------

Thanks for the review. Committed to 0.8 after addressing all issues except #2. 
ISR is used in too many places. It's probably better to clean it up in trunk. 
Otherwise, merging to trunk will be hard.

These are issues that I found from the log during a typical deployment of 
Kafka. Will keep it open for some time in case new critical ones come up.
                
> Cleanup log4j logging
> ---------------------
>
>                 Key: KAFKA-649
>                 URL: https://issues.apache.org/jira/browse/KAFKA-649
>             Project: Kafka
>          Issue Type: Improvement
>    Affects Versions: 0.8
>            Reporter: Jay Kreps
>            Assignee: Jun Rao
>            Priority: Blocker
>         Attachments: kafka-649.patch
>
>
> Review the logs and do the following:
> 1. Fix confusing or duplicative messages
> 2. Assess that messages are at the right level (TRACE/DEBUG/INFO/WARN/ERROR)
> It would also be nice to add a log4j logger for the request logging (i.e. the 
> access log) and another for the controller state change log, since these 
> really have their own use cases.

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

Reply via email to