Dear JC,
Thanks for your comments, I totally like the idea to have 3 subtasks.
It seems that I don’t have access of JBS at present. May I ask your help
for that?
I think the 3 subtask could be:
1. Add dump to file support for jmap –histo
2. Add incremental dump for jmap –histo
3. Add parallel heap iteration for jmap –histo.
Is it reasonable ?
Thanks!
BRs,
Lin
From: JC Beyler [mailto:[email protected]]
Sent: Wednesday, December 19, 2018 12:56 AM
To: 臧琳 <[email protected]>
Cc: [email protected]
Subject: Re: [PATCH] 8214535: Extend JMap to support parallel and incremental
heap scanning
Hi Lin,
Would it not be easier to make 3 different JBS items that are all enhancements?
Or perhaps 3 subtasks to the original JDK-8214535? Then you could send out a
request for review for the first two and we can talk about the third.
I think all three parts can be considered separately (and it seems you do too).
Thanks,
Jc
On Tue, Dec 18, 2018 at 12:58 AM 臧琳 <[email protected]<mailto:[email protected]>>
wrote:
Hi All,
I am preparing the patch for
https://bugs.openjdk.java.net/browse/JDK-8214535, and want to get your
suggestions.
To make the patch easier to review. I plan to make 3 patches as following:
1. one patch for enabling file dump of "jmap
-histo".(http://cr.openjdk.java.net/~xiaofeya/JDK-8214535/file_dump/webrev.00/webrev/)
2. one patch for incremental dump intermediate data to file of "jmap
-histo".(http://cr.openjdk.java.net/~xiaofeya/JDK-8214535/incremental/webrev.00/)
3. one patch for parallel iterating heap of "jmap -histo". (WIP)
And the patches for item 1 & 2 are ready. patch for 3 is WIP.
May I ask your help to review these patches?
Thanks.
BRs,
Lin
--
Thanks,
Jc
