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

JBIDE-18501 - autodetected jvms missing hashcode and equals methods leav... #289

Merged
merged 1 commit into from
Oct 3, 2014

Conversation

robstryker
Copy link
Member

...ing jmx nav full of stale references

}


private JvmKey getJvmKey(IActiveJvm jvm) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not cache this jvmkey instance so we do not trash create them on every call to hascode/key ?

…eaving jmx nav full of stale references

JBIDE-18501 - autodetected jvms missing hashcode and equals methods leaving jmx nav full of stale references
@adietish
Copy link
Member

adietish commented Oct 3, 2014

I tested this and for me I have the following:

EXEC: open JMX Navigator and unfold "Local Processes"
ASSERT: have Eclipse listed
EXEC: launch in cmd line "seq 100 | awk '{ print "/usr/java/jdk1.6.0_45/bin/jps";}' | sh
Result:

  • before the patch:
    3 more nodes showing up that dont go away even when cmd-line finishes
  • after the patch:
    1 more node shows up and goes away once the cmd-line finishes

Seems correcting the issue!

@maxandersen
Copy link
Member

I can confirm it works too, built against 4.2.x. good fix. +1for apply

@robstryker robstryker merged commit 1fcaf6b into jbosstools:master Oct 3, 2014
@robstryker robstryker deleted the JBIDE-18501 branch November 6, 2014 10:19
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

3 participants