RFR: JDK-8306441: Two-stage Segmented Heap Dump [v5]

[Yi Yang]

On Tue, 16 May 2023 18:41:26 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> Hi, can I have a review for this patch?
>
> @y1yang0 Sorry no one has been able to review this so far. The serviceability team is very busy for the next few weeks finishing up JDK 21 changes before RDP1. It's unlikely we'll find time for the review before them.
> 
> I did take a very quick look at the changes just to understand the scope. One thing I noticed that makes this PR hard to review is the code refactoring and relocating that you did. At first it looks like a lot of old code was deleted and a lot of new code added, but in fact most of the new code is just relocated old code. It makes it very hard to tell if there have been any changes to this code. Is there anything you can do to lessen the amount of apparent new code that is actually just moved code?

Hi @plummercj @kevinjwalls @alexmenkov   do you have any plans to review this patch recently? I have addressed some bugs and it passed manual tests& all jtreg tests.

Thanks.

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

PR Comment: https://git.openjdk.org/jdk/pull/13667#issuecomment-1610946603