Looks good!

Thanks,
/Staffan

On 24 sep 2014, at 11:35, Jaroslav Bachorik <jaroslav.bacho...@oracle.com> 
wrote:

> Thanks, Staffan
> 
> On 09/23/2014 08:20 PM, Staffan Larsen wrote:
>> I think we should add some logging to say that we are retrying.
>> 
>> nit: some weird indention on line 623.
> 
> I added logging and improved the retry detection logic a bit (in the previous 
> version the retry counter could have got decremented multiple times during 
> one invocation of 'jcmd' - even though it was very unlikely).
> 
> The weird indentation is gone.
> 
> http://cr.openjdk.java.net/~jbachorik/8057149/webrev.01
> 
> -JB-
> 
>> 
>> Otherwise looks good.
>> 
>> /Staffan
>> 
>> On 23 sep 2014, at 17:59, Jaroslav Bachorik <jaroslav.bacho...@oracle.com> 
>> wrote:
>> 
>>> Please, review this test change
>>> 
>>> Issue : https://bugs.openjdk.java.net/browse/JDK-8057149
>>> Webrev: http://cr.openjdk.java.net/~jbachorik/8057149/webrev.00
>>> 
>>> The test is using 'jcmd' to start/stop the JMX agent dynamically with 
>>> various parameters. Some of the tests use a server socket to simulate 
>>> starting the agent with the desired port not being available. However, it 
>>> seems that in certain situations the agent does not provide a conscious 
>>> error message and fails with RuntimeException instead. This seems to be 
>>> timing related and therefore the solution is to retry the 'jcmd' command a 
>>> few times with a delay to provide some cushion for the timing related 
>>> problems.
>>> 
>>> Thanks,
>>> 
>>> -JB-
>> 
> 

Reply via email to