[11] RFR(XS) 8207761: Split compiler/graalunit/JttReflectFTest.java

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[11] RFR(XS) 8207761: Split compiler/graalunit/JttReflectFTest.java

Ekaterina Pavlova
compiler/graalunit/JttReflectFTest.java still takes too long time to complete and can result in timeout on slow machines.
The change divides this test in two tests.

Also added debug output in test/hotspot/jtreg/compiler/graalunit/common/GraalUnitTestLauncher.java to
help to debug JDK-8206241.

 
     JBS: https://bugs.openjdk.java.net/browse/JDK-8207761
  webrev: http://cr.openjdk.java.net/~epavlova//8207761/webrev.00/index.html
testing: tested by running new tests on the machine on which previous JttReflectFTest.java test did timeout.


Thanks,
-katya

p.s.
  Igor Ignatyev volunteered to sponsor this change.
Reply | Threaded
Open this post in threaded view
|

Re: [11] RFR(XS) 8207761: Split compiler/graalunit/JttReflectFTest.java

Vladimir Kozlov
Looks good.

Thanks,
Vladimir

On 7/18/18 2:10 PM, Ekaterina Pavlova wrote:

> compiler/graalunit/JttReflectFTest.java still takes too long time to
> complete and can result in timeout on slow machines.
> The change divides this test in two tests.
>
> Also added debug output in
> test/hotspot/jtreg/compiler/graalunit/common/GraalUnitTestLauncher.java to
> help to debug JDK-8206241.
>
>
>      JBS: https://bugs.openjdk.java.net/browse/JDK-8207761
>   webrev:
> http://cr.openjdk.java.net/~epavlova//8207761/webrev.00/index.html
> testing: tested by running new tests on the machine on which previous
> JttReflectFTest.java test did timeout.
>
>
> Thanks,
> -katya
>
> p.s.
>   Igor Ignatyev volunteered to sponsor this change.
Reply | Threaded
Open this post in threaded view
|

Re: [11] RFR(XS) 8207761: Split compiler/graalunit/JttReflectFTest.java

Ekaterina Pavlova
Thanks Vladimir!


On 7/18/18 2:22 PM, Vladimir Kozlov wrote:

> Looks good.
>
> Thanks,
> Vladimir
>
> On 7/18/18 2:10 PM, Ekaterina Pavlova wrote:
>> compiler/graalunit/JttReflectFTest.java still takes too long time to complete and can result in timeout on slow machines.
>> The change divides this test in two tests.
>>
>> Also added debug output in test/hotspot/jtreg/compiler/graalunit/common/GraalUnitTestLauncher.java to
>> help to debug JDK-8206241.
>>
>>
>>      JBS: https://bugs.openjdk.java.net/browse/JDK-8207761
>>   webrev: http://cr.openjdk.java.net/~epavlova//8207761/webrev.00/index.html
>> testing: tested by running new tests on the machine on which previous JttReflectFTest.java test did timeout.
>>
>>
>> Thanks,
>> -katya
>>
>> p.s.
>>   Igor Ignatyev volunteered to sponsor this change.