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 225632 - 50s - EDT blocked by search.MatchingObject$DetailsChildren.createNodes() 49,633 ms
Summary: 50s - EDT blocked by search.MatchingObject$DetailsChildren.createNodes() 49,6...
Status: VERIFIED FIXED
Alias: None
Product: utilities
Classification: Unclassified
Component: Search (show other bugs)
Version: 7.3
Hardware: All All
: P2 normal (vote)
Assignee: Jaroslav Havlin
URL:
Keywords: PERFORMANCE
Depends on:
Blocks:
 
Reported: 2013-02-01 12:14 UTC by Alexander Simon
Modified: 2013-03-20 13:21 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter: 198253

Attachments
nps snapshot (264.72 KB, application/nps)
2013-02-01 12:14 UTC, Alexander Simon 		Details
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 Alexander Simon 2013-02-01 12:14:35 UTC 
This bug was originally marked as duplicate of bug 221962, that is already resolved. This bug is still valid, so this seems to be another bug, but it might be related.

Build: NetBeans IDE Dev (Build 20130201-52f7eca05e7c)
VM: Java HotSpot(TM) Client VM, 23.6-b04, Java(TM) SE Runtime Environment, 1.7.0_11-b21
OS: SunOS

User Comments:
alexvsimon: refresh find results

alexvsimon: find in project



Maximum slowness yet reported was 49799 ms, average is 37678
Comment 1 Alexander Simon 2013-02-01 12:14:42 UTC 
Created attachment 130928 [details]
nps snapshot
Comment 2 Jaroslav Tulach 2013-02-04 11:22:55 UTC 
org.netbeans.editor.BaseDocument.render invoked 761 times is slightly too much, I guess.
Comment 3 Jaroslav Havlin 2013-02-04 15:59:42 UTC 
(In reply to comment #2)
> org.netbeans.editor.BaseDocument.render invoked 761 times is slightly too much,
> I guess.
This method is not called for rendering, but for registering Line objects 
(the code is wrapped to a renderer just for synchronization, see BaseDocument.Set.safelyRegisterLine).
If 761 matches are found, this number of invocations should be correct.

http://hg.netbeans.org/core-main/rev/ccb9f9ab5d46
Running TextDetail.prepareLine() in background should fix this bug.
Please verify. Thanks.
Comment 4 Quality Engineering 2013-02-05 12:36:55 UTC 
Integrated into 'main-golden', will be available in build *201302050917* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/ccb9f9ab5d46
User: Jaroslav Havlin <jhavlin@netbeans.org>
Log: #225632: 50s - EDT blocked by search.MatchingObject$DetailsChildren.createNodes()
Comment 5 Marian Mirilovic 2013-03-07 07:49:51 UTC 
Alexander,
are you able to reproduce this issue in daily build ? If not, we will backport the fix into NB 7.3 Patch 1. 
Thanks in advance.
Comment 6 Alexander Simon 2013-03-07 08:45:56 UTC 
(In reply to comment #5)
> Alexander,
> are you able to reproduce this issue in daily build ? If not, we will backport
> the fix into NB 7.3 Patch 1. 
> Thanks in advance.

Apparently, problem went away.
Comment 7 Marian Mirilovic 2013-03-07 08:47:16 UTC 
(In reply to comment #6)
> (In reply to comment #5)
> > are you able to reproduce this issue in daily build ? If not, we will backport the fix into NB 7.3 Patch 1. 
> Apparently, problem went away.

So verified, thanks a lot !
Comment 8 Jaroslav Havlin 2013-03-07 13:25:39 UTC 
Integrated into release73:
http://hg.netbeans.org/releases/rev/1069e1bd9a93 (graft)
http://hg.netbeans.org/releases/rev/f920828317e2 (spec. version, long desc.)
Comment 9 Quality Engineering 2013-03-11 01:13:35 UTC 
Integrated into 'releases', will be available in build *201303102200* or newer. Wait for official and publicly available build.
Changeset: http://hg.netbeans.org/releases/rev/1069e1bd9a93
User: Jaroslav Havlin <jhavlin@netbeans.org>
Log: #225632: 50s - EDT blocked by search.MatchingObject$DetailsChildren.createNodes()
Comment 10 Marian Mirilovic 2013-03-20 13:21:44 UTC 
verified by checking the correct changesets are available in release73 branch