Looks good to me,

Misha

On 12/4/18, 5:41 PM, David Holmes wrote:
Hi Leonid,

This seems fine to me. It's been some time since hotspot hosted any of the native code used by networking.

Thanks,
David

On 5/12/2018 7:36 am, Leonid Mesnik wrote:
Hi

Could you please review following fix which removes vmTestbase/nsk/stress/network tests. These tests are pretty old stress networking tests and are don't seem to be useful for hotspot. I was not able to find any product bugs related with these tests failures.

Bug JDK-4245704 <https://bugs.openjdk.java.net/browse/JDK-4245704> which is mentioned in the test sources is not related with these tests itself.

If anyone from network team consider these tests useful then they could be just moved in more appropriate location.

I verified that there are no additional changes in TEST.groups file are required.

webrev: http://cr.openjdk.java.net/~lmesnik/8210107/webrev.00/ <http://cr.openjdk.java.net/~lmesnik/8210107/webrev.00/> bug: https://bugs.openjdk.java.net/browse/JDK-8210107 <https://bugs.openjdk.java.net/browse/JDK-8210107>

Leonid

Reply via email to