Looks good! Just don't forget to add "noreg-self" label to the issue.
-JB- On 13.1.2015 10:28, shanliang wrote:
Hi Please review this test bug fix Bug: https://bugs.openjdk.java.net/browse/JDK-8068774 Webrev: http://cr.openjdk.java.net/~sjiang/JDK-8068774/00/ The problem must be here: 98 monitorProxy.start(); 99 100 final int initGetCount = observedProxy.getGetCount(); The test calls initGetCount after starting the monitor, but the test case 1 is: "Remove monitored MBean within monitored getAttribute" that means if the monitor calls getAttribute before the test calls observedProxy.getGetCount(), then no more getAttribute will happen and the return of observedProxy.getGetCount() will not be changed any more. This is why the test is timeout. I reproduced the bug by inserting at line 99: Thread.sleep(1000); Thanks, Shanliang
