Skip to content
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

fix delete topic incompletion #182

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

jeanlyn
Copy link

@jeanlyn jeanlyn commented Dec 16, 2015

We don't need to delete the path under /brokers/topics when we deleting the topic because kafka will help us to do this, otherwise the delete process will hung, and the next time we can not create the same topic.

@jeanlyn
Copy link
Author

jeanlyn commented Dec 16, 2015

The test failure information:

 - delete topic *** FAILED ***
  Vector("cm-test", "cm-unit-test") contained "cm-unit-test" Failed to delete topic (TestClusterManagerActor.scala:243)

The root cause is the path under /broker/topics will not delete without the real kafka cluster. Hence, can we remove this check?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant