-
Notifications
You must be signed in to change notification settings - Fork 626
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Kafka-node does not recover from UnknownTopicOrPartition error (when topic has been reassigned to a different broker) #319
Comments
This problem also happens when I manually delete a topic. |
@hyperlink Even though your code does prevent kakfa-node from going into an endless error loop, but for some reason it actually prevents a topic from being deleted on the broker. I'm using kafka-manager to manually delete topics. |
It seems like the fix only handles the I'm encountering this problem when reassigning partitions. When I'm producing I get the |
@itamarwe do you mind create an issue for this? Also PRs are welcome if you want to fix this. |
I encountered this problem too. |
This issue happened in our production environment running version
0.2.29
. Devops tried to balance the load across our kafka cluster by moving some topics around. I would expect this also to be an issue in0.3.1
as well.Test setup
Please see issue #277 for details on how to set up your environment for this test.
Update
docker-compose.yml
:Add
move.json
Steps to reproduce
./kafkatest.js
docker-compose scale kafka=3
./kafka-topics.sh --zookeeper 192.168.99.100:2181 --topic KafkaConnectivityTest --describe
to verify the topic is currently not on leader you are changing to otherwise you will need to update thereplicas
field inmove.json
to a different broker./kafka-reassign-partitions.sh --zookeeper 192.168.99.100:2181 --reassignment-json-file move.json --execute
Comments
According to the error description on the kafka protocol page
This request is for a topic or partition that does not exist on this broker.
This appears to be recoverable error. I'm currently in the process of working on a PR to fix this issue.The text was updated successfully, but these errors were encountered: