On Wed, 21 Apr 2021 18:40:16 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:

>>> My concerns with your proposed testing is that it always targets the same 
>>> application with the same heap, and does not read/process the hprof file to 
>>> make sure it is not corrupt.
>> 
>> Hmm, I agree, I will do more test and update here. Thanks!
>
>> > My concerns with your proposed testing is that it always targets the same 
>> > application with the same heap, and does not read/process the hprof file 
>> > to make sure it is not corrupt.
>> 
>> Hmm, I agree, I will do more test and update here. Thanks!
> 
> As for jtreg testing, we have a few heap dumping related tests. I'm not sure 
> how good they are. It would be good to understand what testing we currently 
> do.
> 
> In addition to jtreg testing, you might want to try just launching some large 
> java apps (netbeans and intellij come to mind), dump their heaps, and then 
> process them with some existing tool. I'm not suggesting this be part of 
> regular testing, but just a sanity check you do on your own before pushing 
> the changes.

Hi Chris, @plummercj 
     Here are the tests I have conducted:
     - tier1, tier2,tier3 all passed
     - netbeans with heap usage of 500mb, 1000mb, plus "randomly do some 
operation and dump the heap".  tested 30 times and all passed.
     - a workload that generate different object to fill heap, I have tested 
with heap usage from 1GB to 8GB.  All passed.
     All the scenario are tested with/without the gz option. 
     Do you think these tests are ok?

BRs,
Lin

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

PR: https://git.openjdk.java.net/jdk/pull/2261

Reply via email to