Sure David.  Thanks.

Thanks,
Sharath


-----Original Message-----
From: David Holmes 
Sent: Tuesday, November 21, 2017 10:53 AM
To: Sharath Ballal; Jini Susan George; serviceability-dev@openjdk.java.net
Subject: Re: RFR: JDK-8191401 - [TESTBUG] serviceability/sa/ClhsdbFlags.java 
can fail due to UseJVMCICompiler

Sorry I missed those commercial flags.

But we need this fix in today else the test will be problem listed.

Don't swap those flags for anything just delete them. Then no extra testing 
needed.

Thanks,
David

On 21/11/2017 3:16 PM, Sharath Ballal wrote:
> OK.  Thanks Jini.
> 
> 
> Thanks,
> Sharath
> 
> 
> -----Original Message-----
> From: Jini George
> Sent: Tuesday, November 21, 2017 10:43 AM
> To: Sharath Ballal; serviceability-dev@openjdk.java.net
> Subject: Re: RFR: JDK-8191401 - [TESTBUG] 
> serviceability/sa/ClhsdbFlags.java can fail due to UseJVMCICompiler
> 
> Hi Sharath,
> 
> Since StartFlightRecording and UseAppCDS are closed only flags currently, 
> these would fail for testing with open only builds. It might be prudent to 
> remove the checking of these flags too. Other than this, the changes look 
> good.
> 
> Thank you,
> Jini.
> 
> On 11/20/2017 2:54 PM, Sharath Ballal wrote:
>> Hello,
>>
>> Pls review changes for the following issue:
>>
>> Bug ID: https://bugs.openjdk.java.net/browse/JDK-8191401
>>
>> Webrev: http://cr.openjdk.java.net/~sballal/8191401/webrev.00/
>>
>> Thanks,
>>
>> Sharath
>>

Reply via email to