On Thu, 7 Sep 2023 15:58:26 GMT, Martin Buchholz <mar...@openjdk.org> wrote:

>> Soumadipta Roy has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Remove extra new line
>
> test/jdk/java/util/concurrent/tck/JSR166TestCase.java line 45:
> 
>> 43:  * @modules java.management
>> 44:  * @run junit/othervm/timeout=1000 JSR166TestCase
>> 45:  * @run junit/othervm/timeout=1000 -Djava.security.manager=allow 
>> JSR166TestCase
> 
> security manager testing is relatively less important.   I would move this to 
> a lower tier, while moving a test with 
> -Djsr166.testImplementationDetails=true into tier1

Hi Martin,

I will raise another commit trying to add summary for the segregations

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15619#discussion_r1318854380

Reply via email to