RFR: 7879: Automated Analysis taking very long time to produce results for Class Leak Rule and showing wrong results. [v2]
Suchita Chaturvedi
schaturvedi at openjdk.org
Thu Oct 6 04:19:30 UTC 2022
On Wed, 5 Oct 2022 20:16:36 GMT, Brice Dutheil <bdutheil at openjdk.org> wrote:
>> @Gunde @bric3 I have tried to implement all the review comments. Please re-review the PR and let me know if there is something which is still need to be corrected.
>
> @Suchitainf There's still these two comment to address in my opinion:
> * https://github.com/openjdk/jmc/pull/419#discussion_r987769827
> * https://github.com/openjdk/jmc/pull/419#discussion_r988313757
>
> Note I'm really unsure about the result ordering there, may be @thegreystone or @Gunde may chime in there to make sure this change is ok.
@bric3 We need to show the names of top 5 classes which were loaded maximum times without being unloaded. So the ordering matters here. That's why I have skipped that comment.
-------------
PR: https://git.openjdk.org/jmc/pull/419
More information about the jmc-dev
mailing list