GitHub user SinghAsDev opened a pull request:

    https://github.com/apache/kafka/pull/1609

    Minor: Improve PartitionState logging and remove duplication of code

    Currently, logs involving PartitionState are not very helpful.
    
    ```
        Broker 449 cached leader info 
org.apache.kafka.common.requests.UpdateMetadataRequest$PartitionState@3285d64a 
for partition prod-guts-out-main-shunt-23 in response to UpdateMetadata request 
sent by controller 356 epoch 138 with correlation id 0
    
        TRACE state.change.logger: Broker 449 received LeaderAndIsr request 
org.apache.kafka.common.requests.LeaderAndIsrRequest$PartitionState@66d6a8eb 
correlation id 3 from controller 356 epoch 138 for partition 
[prod-guts-out-main-shunt,148]
    ```
    


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/SinghAsDev/kafka partitionState

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/kafka/pull/1609.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1609
    
----
commit 7a99fb2fb4531039bc195f870f84cdfe1358dfb2
Author: Ashish Singh <asi...@cloudera.com>
Date:   2016-07-11T20:31:29Z

    Minor: Remove duplicated code for PartitionState and add toString override 
for better logging.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to