On 13/01/15 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/

Looks good Shanliang!

best regards,

-- daniel



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

Reply via email to