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 191509 - Performance regression in comparison with 6.9.1
Summary: Performance regression in comparison with 6.9.1
Status: RESOLVED FIXED
Alias: None
Product: cnd
Classification: Unclassified
Component: Code Model (show other bugs)
Version: 7.0
Hardware: All All
: P3 normal (vote)
Assignee: Alexander Simon
URL:
Keywords: PERFORMANCE
: 178655 (view as bug list)
Depends on: 192396
Blocks:
  Show dependency tree
 
Reported: 2010-11-01 09:59 UTC by Alexander Pepin
Modified: 2011-05-27 08:31 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Pepin 2010-11-01 09:59:52 UTC
According to performance test results now we have a regression in response time in comparison with CND in NB 6.9.1. Moreover since October 27 we have an additional regression. Please analyze the cause of the regressions and fix them by FCS.
Comment 1 Alexander Simon 2010-11-02 19:46:00 UTC
Some investigation:
regression 5% in CndUtils.assertNormalized()
regression 2% is CndLexerUtilities.isCppLanguage()
Comment 2 Alexander Simon 2010-11-02 20:10:48 UTC
fixed CndLexerUtilities.isCppLanguage() in change set:
http://hg.netbeans.org/cnd-main/rev/3a047d680a40

disabled assertion CndUtils.assertNormalized() called from FileContainer.getFileKey() in change set:
http://hg.netbeans.org/cnd-main/rev/15d3508e67d4
Vladimir K: Invent another way to check normalized file without such huge fare.
Comment 3 Quality Engineering 2010-11-03 03:55:50 UTC
Integrated into 'main-golden', will be available in build *201011030000* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/3a047d680a40
User: alexvsimon@netbeans.org
Log: fixing Bug #191509 Performance regression in comparison with 6.9.1
Comment 4 Alexander Simon 2010-11-03 11:33:16 UTC
Alexander Pepin:
Does performance tests use "-da" Java option?
Make sure that performance test disable assertions.
Comment 5 Alexander Simon 2010-11-24 19:28:41 UTC
Change sets:
http://hg.netbeans.org/cnd-main?cmd=changeset;node=24a5888212f5
http://hg.netbeans.org/cnd-main?cmd=changeset;node=c98037780773
http://hg.netbeans.org/cnd-main?cmd=changeset;node=95436e119e6c
http://hg.netbeans.org/cnd-main?cmd=changeset;node=53f34c5bc82b
improve performance of reference resolving:
- do not search in namespaces in case pure C source file
- remove unneeded getCsmFile()
- improve performance of FileElementsCollector
- improve search in project enumerators
Comment 6 Quality Engineering 2011-01-27 08:02:28 UTC
Integrated into 'main-golden', will be available in build *201101270001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/47d626d9cb4d
User: Alexander Simon <alexvsimon@netbeans.org>
Log: fixing Bug #191509 Performance regression in comparison with 6.9.1
- refactor Resolver3: remove redundant gather maps for local context.
Comment 7 nnnnnk 2011-01-28 20:15:06 UTC
*** Bug 178655 has been marked as a duplicate of this bug. ***
Comment 8 Quality Engineering 2011-04-30 04:38:22 UTC
Integrated into 'main-golden', will be available in build *201104300000* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/99ff19e3f2be
User: Alexander Simon <alexvsimon@netbeans.org>
Log: fixing Bug #191509 Performance regression in comparison with 6.9.1
- reduce soft memory leaks
Comment 9 Alexander Simon 2011-05-27 08:31:00 UTC
It seems there are no obvious performance regressions now.