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