[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-hotspot-runtime-dev
Subject: Re: RFR(XS): 8176403: [TESTBUG] gc/arguments/TestSmallInitialHeapWithLargePageAndNUMA is referencing
From: sangheon <sangheon.kim () oracle ! com>
Date: 2017-03-08 23:43:14
Message-ID: 960cc068-cd6a-6a02-c07b-eca10cce5df7 () oracle ! com
[Download RAW message or body]
Hi Dan,
On 03/08/2017 03:37 PM, Daniel D. Daugherty wrote:
> On 3/8/17 4:34 PM, Daniel D. Daugherty wrote:
>> On 3/8/17 4:31 PM, David Holmes wrote:
>>> Hi Sangheon,
>>>
>>> On 9/03/2017 9:24 AM, sangheon wrote:
>>>> Hi all,
>>>>
>>>> Could I have some reviews for this change?
>>>>
>>>> As current TestSmallInitialHeapWithLargePageAndNUMA.java uses old test
>>>> group tag, changed to use the new tag.
>>>
>>> To be clear it was not a "test group tag" but a library path that is
>>> wrong in the test and which has been fixed.
>>
>> But you can have a second (R)eview if you need it.
>
> Wow! That could have been more clear...
>
> Thumbs up!
Thanks for the review.
Thanks,
Sangheon
>
> Dan
>
>
>>
>> Dan
>>
>>
>>>
>>>> The test uses the old test group tag but it is not tested on JPRT.
>>>> Because recently jprt.properties file was changed and I was using the
>>>> old one.
>>>>
>>>> CR: https://bugs.openjdk.java.net/browse/JDK-8176403
>>>> webrev: http://cr.openjdk.java.net/~sangheki/8176403/webrev.0
>>>
>>> Looks fine to me.
>>>
>>> I consider this small enough to be pushed under the trivial change
>>> rule - particularly as it blocks other pushes.
>>>
>>> Thanks,
>>> David
>>>
>>>
>>>> Testing: JPRT (checked the test is ran and passed)
>>>>
>>>> Thanks,
>>>> Sangheon
>>
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic