Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 365841

Summary: [performance] Performance regression in CleanUpPerfTest#testCodeStyleCleanUp()
Product: [Eclipse Project] JDT Reporter: Satyam Kandula <satyam.kandula>
Component: UIAssignee: JDT-UI-Inbox <jdt-ui-inbox>
Status: CLOSED WONTFIX QA Contact:
Severity: normal    
Priority: P3 CC: daniel_megert, markus.kell.r
Version: 3.8Keywords: performance
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard: stalebug
Bug Depends on: 374441    
Bug Blocks:    

Description Satyam Kandula CLA 2011-12-07 04:15:00 EST
There is a <10% performance regression in CleanUpPerfTest#testCodeStyleCleanUp(). I see this on both linux and windows machines.
Comment 1 Dani Megert CLA 2011-12-07 04:46:21 EST
Deepak, please investigate.
Comment 4 Dani Megert CLA 2012-01-12 11:39:54 EST
> This were OK till 3.7.1 and I do not see anything that has changed in this
> area...

The "..." mean, you're investigating deeper now, right?
Comment 5 Deepak Azad CLA 2012-01-12 11:56:16 EST
> The "..." mean, you're investigating deeper now, right?
Yes.
Comment 6 Deepak Azad CLA 2012-05-22 04:36:32 EDT
I ran the test with all milestone builds, there looks to be a slowdown in M3 and then a slight improvement in M6

3.7
Elapsed Process:         968ms        (95% in [892ms, 1.04s])        Measurable effect: 134ms (1.3 SDs) (required sample size for an effect of 5% of mean: 78)

M2
Elapsed Process:         970ms        (95% in [896ms, 1.04s])        Measurable effect: 129ms (1.3 SDs) (required sample size for an effect of 5% of mean: 72)

M3
Elapsed Process:        1.04s         (95% in [927ms, 1.14s])        Measurable effect: 191ms (1.3 SDs) (required sample size for an effect of 5% of mean: 137)

M6
Elapsed Process:         996ms        (95% in [901ms, 1.09s])        Measurable effect: 166ms (1.3 SDs) (required sample size for an effect of 5% of mean: 112)

As I mentioned in comment 3 I do not see anything that changed in JDT UI which would explain this. 

My best guess at this point are the new static analysis features introduced in this release. More specifically, resource leak warnings were added in M3 (bug 349326), the analysis requires computation of type hierarchies of (almost) all types encountered in code. Also in M3 some of the new code was executed all the time i.e. even when the resource leak options were off, this was improved a little during M6.
Comment 7 Dani Megert CLA 2013-01-28 05:13:01 EST
No action required until bug 374441 is fixed.
Comment 8 Eclipse Genie CLA 2020-04-17 14:41:18 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.