Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

Doctor Kafka generated invalid reassignment #18

Closed
daywithoutrain opened this issue May 7, 2018 · 4 comments
Closed

Doctor Kafka generated invalid reassignment #18

daywithoutrain opened this issue May 7, 2018 · 4 comments

Comments

@daywithoutrain
Copy link

We adapted to use DoctorKafka in our production system recently. However, we found that the first time after using doctorkafka, a broker host having some issues and resulted in many under replicated partitions. doctorkafka tried to resolve under replicated partitions, but it somehow generated an reassignment including the following entries: (the entries are copied from doctor kafka's log)

 {
            "topic": "mydomain.MyTopic1",
            "partition": 40,
            "replicas": [
                65633,
                65633,
                65633
            ]
        },
{
            "topic": "mydomain.MyTopic2",
            "partition": 2,
            "replicas": [
                65633,
                65633,
                65633
            ]
        },

...
This impacted our production system and delayed the downstream process. We have more than 50 broker hosts and 1 host was having trouble. I assume that the above assignment was generated due to a bug, is that correct? Have you seen this issue before? I'd appreciate if you could look into it ASAP. Thanks!

@yuyang08
Copy link
Contributor

What is the replication factor for your topics? DoctorKafka has a known issue w.r.t. handling topics of replicationFactor = 1. If you set replication factor > 1, DoctorKafka should handle the topic assignment fine.

@daywithoutrain
Copy link
Author

daywithoutrain commented May 10, 2018 via email

@yuyang08
Copy link
Contributor

There might be a bug somewhere. Can you share the assignment plan by doctorkafka from $your_doctorkafka_site/servlet/actions, and the related doctorkafka logs?

@daywithoutrain
Copy link
Author

daywithoutrain commented May 11, 2018 via email

LKCCYh0woz added a commit to LKCCYh0woz/pinterestf that referenced this issue Dec 10, 2021
Fix the invalid assignment issue that is reported in pinterest/DoctorK#18, and add a unit test for validation
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants