This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 213072 - [regression] find usages is slower comparing with 7.0.1
Summary: [regression] find usages is slower comparing with 7.0.1
Status: RESOLVED FIXED
Alias: None
Product: cnd
Classification: Unclassified
Component: Code Model (show other bugs)
Version: 7.2
Hardware: All All
: P2 normal (vote)
Assignee: Egor Ushakov
URL:
Keywords:
Depends on: 202681
Blocks:
  Show dependency tree
 
Reported: 2012-05-28 08:24 UTC by Vladimir Voskresensky
Modified: 2012-06-09 04:35 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Vladimir Voskresensky 2012-05-28 08:24:40 UTC 
Alexander's test shows regression between 7.1 code base and trunk. Needs investigation
Comment 1 Egor Ushakov 2012-05-28 15:32:55 UTC 
Unable to reproduce, please remeasure
Comment 2 Egor Ushakov 2012-05-29 15:53:09 UTC 
regression in comparison with 7.0.1:

current trunk with 500mb and 4 threads:
Unresolved 913 (0.16%) where MacroBased 831 (0.15%) of 561696 checkpoints [TemplateBased warnings 2 (0.00%), Builtin 0 (0.00%)]
Line count: 545114, time 222629 ms, 
speed 2448.53 lines/sec, 2523.01 refs/sec
Analyzing ccfe_701 took 222678ms

NB 7.0.1 with 500mb and 4 threads:
Unresolved 953 (0.17%) where MacroBased 831 (0.15%) of 561696 checkpoints [TemplateBased warnings 2 (0.00%), Builtin 0 (0.00%)]
Line count: 545190, time 132401 ms, 
speed 4117.72 lines/sec, 4242.39 refs/sec
Analyzing ccfe_701 took 132449ms
Comment 3 Egor Ushakov 2012-05-29 16:09:38 UTC 
it looks like we're consuming much more memory and thus spend too much time in
GC. From the snapshot it seems that fix for the bug 202681 caused that:
probably notifyClosed is not being called during find usages and the cache in
CppEditorSupportProvider holds CppEditorSupport that has a reference to
DataObject somewhere inside and so never gets GCed.
Backing out this fix brings memory usage and performance to normal.
Comment 4 Egor Ushakov 2012-05-31 14:25:03 UTC 
performance loss with 1Gb memory caused by the C++11 support introduction in http://hg.netbeans.org/cnd-main/rev/214342 and is predictable.
Small improvement in:
http://hg.netbeans.org/cnd-main/rev/f805f9251023
keeping as a P3 until we find out what to do with the bug 202681
Comment 5 Vladimir Voskresensky 2012-05-31 14:27:35 UTC 
let's keep it as P2 to be on radar; memory leak is rather big and either bug #202681 has to be fixed or we need workaround again.
Comment 6 Egor Ushakov 2012-06-08 13:05:27 UTC 
#202681 is now fixed, so this one can be marked as fixed too
Comment 7 Quality Engineering 2012-06-09 04:35:25 UTC 
Integrated into 'main-golden', will be available in build *201206090001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/f805f9251023
User: Egor Ushakov <gorrus@netbeans.org>
Log: fixing #213072 - [regression] find usages is slower comparing with 7.0.1