Code review request, 8037557 test SessionCacheSizeTests.java timeout

Xuelei Fan xuelei.fan at oracle.com
Thu Apr 10 23:10:31 UTC 2014


On 4/10/2014 10:51 PM, Weijun Wang wrote:
> So you want to see which line throws SocketTimeoutException?
> 
the line calling accept() and read().

> What does that thread.join(120000) do? Is it possible that after it
> finishes no exception was thrown yet and your test shown as succeeded?
> 
IO exception should be thrown when the client cannot read and write.

Thanks,
Xuelei

> --Max
> 
> On 4/10/2014 20:45, Xuelei Fan wrote:
>> On 4/10/2014 8:42 PM, Weijun Wang wrote:
>>> The code change looks harmless, but I am not sure how it could provide
>>> more information next time.
>>>
>> Next time, with accept timeout and read timeout, we would know on which
>> point the threads is more easier to get timeout.
>>
>> Xuelei
>>
>>> --Max
>>>
>>>
>>> On 4/10/2014 20:17, Xuelei Fan wrote:
>>>> Hi,
>>>>
>>>> Please review this test case update:
>>>>       http://cr.openjdk.java.net/~xuelei/8037557/webrev.00/
>>>>
>>>> This is a test timeout issue.  Cause is still unknown.  I guess that
>>>> one
>>>> or more of the server threads had not yet been scheduled to run by the
>>>> system within the timeout of the testing.  I added more timeout in the
>>>> code.  Hope next time timeout failure, we can get more information.
>>>>
>>>> Thanks,
>>>> Xuelei
>>>>
>>




More information about the security-dev mailing list