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

Joel Koshy commented on KAFKA-1350:
-----------------------------------

[~guozhang] I'm not sure I follow your question. I meant that since we use the 
logger API directly for state-change.log, we need to check if 
(stateChangeLogger.is*Enabled).

Also, are you sure you recompiled your test - the final count at the INFO level 
should be 3. (I tried that locally as well.)


> Fix excessive state change logging
> ----------------------------------
>
>                 Key: KAFKA-1350
>                 URL: https://issues.apache.org/jira/browse/KAFKA-1350
>             Project: Kafka
>          Issue Type: Bug
>    Affects Versions: 0.8.1
>            Reporter: Joel Koshy
>            Assignee: Neha Narkhede
>            Priority: Blocker
>             Fix For: 0.8.1.1
>
>         Attachments: KAFKA-1350.patch, KAFKA-1350_2014-03-29_23:28:07.patch
>
>
> I can provide steps to reproduce this issue.  The state change logger needs
> to be guarded (to check if trace logging is turned on or not).
> The delete topic patch significantly increased the amount of logging that we
> do both on the controller. This results in higher latencies in state
> transitions and can slow down the controller (as well as the broker).  This
> slow-down was how we ran into KAFKA-1342.



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

Reply via email to