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.

Sign up for GitHub

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

Jump to bottom

Clean up mbean removal #67

Closed
3 tasks done
mk23 opened this issue Aug 27, 2015 · 0 comments
Closed
3 tasks done

Clean up mbean removal #67

mk23 opened this issue Aug 27, 2015 · 0 comments
Assignees
@mk23
Labels
enhancement
Milestone
jmxproxy.3.3.0

Comments

@mk23
Copy link
Owner

mk23 commented Aug 27, 2015

Fix these issues introduced in #66.

  • rename removeMBeam() to removeMBean()
  • use Set.removeAll() to determine which mbeans are stale
  • add removeMBean test
@mk23 mk23 self-assigned this Aug 27, 2015
@mk23 mk23 added this to the jmxproxy.3.2.1 milestone Aug 27, 2015
@mk23 mk23 assigned mk23 and unassigned mk23 Aug 27, 2015
mk23 added a commit that referenced this issue Sep 6, 2015
@mk23
Issue #67: Rename Host.removeMBeam() to Host.removeMBean().
94abcb1
mk23 added a commit that referenced this issue Sep 6, 2015
@mk23
Issue #67: Use Set.removeAll() to determine stale mbeans.
4518e9e
mk23 added a commit that referenced this issue Sep 6, 2015
@mk23
Issue #67: Add mbean removal test.
2407fca
@mk23 mk23 closed this as completed Sep 6, 2015
mk23 added a commit that referenced this issue Sep 6, 2015
@mk23
Issue #67: Fix stash merge detritus.
53b7732
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mk23 mk23

Labels
enhancement
Projects
None yet
Milestone
jmxproxy.3.3.0
Development

No branches or pull requests
1 participant
@mk23