RFR(S) : 8252004 : remove usage of PropertyResolvingWrapper in vmTestbase/nsk/sysdict
Igor Ignatyev
igor.ignatyev at oracle.com
Mon Aug 31 21:51:08 UTC 2020
Hi Gerard,
thanks for your review.
8219408 is to update tests residing in test/jdk, hotspot tests are going to be updated by 8219140's sub-tasks, which include 8252004 and 8252002. I am not updating all hotspot tests by one patch, b/c in some cases, the tests need small modification to work w/o PropertyResolvingWrapper and splitting the work by test groups make it easier to both work on and review such patches.
Cheers,
-- Igor
> On Aug 31, 2020, at 2:12 PM, gerard ziemski <gerard.ziemski at oracle.com> wrote:
>
> hi Igor,
>
> Looks fine as far a I can tell, however, other tests are waiting till JDK-8219408 to re-enable allowSmartActionnArgs - why don't we need to wait for that here as well?
>
> I noticed that there are other issues filed to address the same issue in other tests, of which there are many (i.e. JDK-8252002) - they will be following the same pattern as this one?
>
>
> cheers
>
>
> On 8/31/20 2:02 PM, Igor Ignatyev wrote:
>> ping?
>>
>> -- Igor
>>
>>> On Aug 21, 2020, at 10:46 AM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
>>>
>>> http://cr.openjdk.java.net/~iignatyev/8252004/webrev.00/
>>>> 22 lines changed: 0 ins; 22 del; 0 mod;
>>>
>>> Hi all,
>>>
>>> could you please review this small patch which removes usage of PropertyResolvingWrapper from nsk/sysdict tests and reenables allowSmartActionArgs?
>>>
>>> background from the main bug:
>>>> CODETOOLS-7902352 added support of using ${property} in action directive, so PropertyResolvingWrapper isn't needed anymore and can be removed.
>>>
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8252004
>>> webrev: http://cr.openjdk.java.net/~iignatyev/8252004/webrev.00
>>> testing: :vmTestbase_nsk_sysdict
>>>
>>> Thanks,
>>> -- Igor
>>>
>
More information about the hotspot-runtime-dev
mailing list