Dear All, 
     I have updated the CSR at https://bugs.openjdk.java.net/browse/JDK-8222319
     May I ask your help to review it? 
     When it is finalized, I will refine the webrev.

BRs,
Lin

> Dear Serguei,
>          Thanks a lot for your reviewing.
> 
> 
> 
> >   System.err.println("      incremental dump support:");
> > +        System.err.println("        chunkcount=<N>    object number 
> > counted (in Kilo) to trigger incremental dump");
> > +        System.err.println("        maxfilesize=<N>   size limit of 
> > incremental dump file (in KB)");
> >
> >
> > From this description is not clear at all what does the chunkcount mean.
> > Is it to define how many heap objects are dumped in one chunk?
> > If so, would it better to name it chunksize instead where chunksize is 
> > measured in heap objects?
> > Then would it better to use the same units to define the maxfilesize as 
> > well?
> > (I'm not insisting on this, just asking.)
>
> The original meaning of  “chunkcount"  is how many objects are dumped in one 
> chunk, and the “maxfilesize” is the limited size of the dump file.
> For example, “chunkcount=1, maxfilesize=10” means that intermediated data 
> will be written to the dump file for every 1000 objects, and
> when the dump file is larger than 10k,erase the file and rewrite it with the 
> latest dumped data.
>
> The reason I didn’t use object count to control the dump file size is that 
> there can be humongous object, which may cause the file too large.
> Do you think use object size instead of chunkcount is a good option? So the 
> two options can be with same units.

> BRs,
> Lin

Reply via email to