Skip to content
This repository has been archived by the owner on Oct 6, 2018. It is now read-only.

Fixes #36. Fixes #30. #40

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

Fixes #36. Fixes #30. #40

wants to merge 14 commits into from

Conversation

ibanner56
Copy link

There's been an issue with web-console-consumer-XXXX nodes not being deleted after you leave a topic's page. I changed how the function was calling delete on the nodes so that it doesn't trigger an eventThread shutdown prior to the delete completing on the parent node. Now you don't end up with undeleted nodes in zookeeper.

@ibanner56 ibanner56 changed the title Modified delete workflow to prevent incomplete deletes. Modified delete workflow to prevent incomplete deletes. Fixes #36. Nov 13, 2014
@ibanner56 ibanner56 changed the title Modified delete workflow to prevent incomplete deletes. Fixes #36. Fixes #36. Fixes #30. Dec 9, 2014
@ibanner56
Copy link
Author

Not all the client connections to kafka were being closed, only the one that was being used. Changed the client closing line (now line 80) to close all the clients that are being opened in getPartitionsLogSize.

@ibanner56 ibanner56 changed the title Fixes #36. Fixes #30. Fixes #36. Fixes a part, but not all of #30. Dec 9, 2014
@ibanner56 ibanner56 changed the title Fixes #36. Fixes a part, but not all of #30. Fixes #36. Fixes #30. Dec 17, 2014
@ches
Copy link

ches commented Mar 25, 2015

Six months and no review from maintainer? Thanks Isaac, will be testing your fork shortly.

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

Successfully merging this pull request may close these issues.

2 participants