RFR: 7879: Automated Analysis taking very long time to produce results for Class Leak Rule and showing wrong results. [v4]
Alex Macdonald
aptmac at openjdk.org
Mon Oct 24 13:48:57 UTC 2022
On Thu, 6 Oct 2022 20:28:14 GMT, Suchita Chaturvedi <schaturvedi at openjdk.org> wrote:
>> This PR takes care of optimizing and correcting the Class leak rule results on Automated Analysis Page.
>
> Suchita Chaturvedi has updated the pull request incrementally with one additional commit since the last revision:
>
> Adding higher timeout as default value
I think it looks okay for me. It would be nice to have a way of reproducing the issue to verify, but I get it's tough if the original jfr is from a customer and can't be shared.
Having said that, @Gunde and @bric3 did substantial review on this PR as well as the previous (https://github.com/openjdk/jmc/pull/248), and while on the surface it looks like everything has been addressed I would feel much more comfortable if they could just confirm that what they wanted addressed was updated in this PR before merging it.
core/org.openjdk.jmc.flightrecorder.rules.jdk/src/main/java/org/openjdk/jmc/flightrecorder/rules/jdk/util/DefaultIItemResultSet.java line 84:
> 82: private void calculateData(IItemCollection input, int configuredTimeout) throws InterruptedException {
> 83: input = input.apply(query.getFilter());
> 84: final IItemCollection newInput = input;
Very minor nit, but this could be moved to be inside the else block where it's used? It's not required for the first chunk of the if-statement.
-------------
PR: https://git.openjdk.org/jmc/pull/419
More information about the jmc-dev
mailing list