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 32824 - Looong time to display Generic VCS mounting wizard.
Summary: Looong time to display Generic VCS mounting wizard.
Status: VERIFIED FIXED
Alias: None
Product: obsolete
Classification: Unclassified
Component: vcscore (show other bugs)
Version: -S1S-
Hardware: PC Windows ME/2000
: P2 blocker (vote)
Assignee: Martin Entlicher
URL:
Keywords: PERFORMANCE
Depends on:
Blocks:
 
Reported: 2003-04-11 11:00 UTC by Jiri Kovalsky
Modified: 2003-07-08 13:04 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Several thread dumps taken during wizard invocation. (83.85 KB, text/plain)
2003-04-11 11:44 UTC, Jiri Kovalsky
Details
The binary patch. Put into <NB-install>/modules/autoload/patches/org-netbeans-modules-vcscore/ folder. (13.14 KB, application/octet-stream)
2003-04-14 12:17 UTC, Martin Entlicher
Details
The contextual diff, that fix this problem. (7.40 KB, patch)
2003-04-14 12:21 UTC, Martin Entlicher
Details | Diff
Here are those exceptions ... (3.93 KB, text/plain)
2003-04-14 13:16 UTC, Jiri Kovalsky
Details
Hm, probably this is what I should attach, I apologize ... :-( (4.76 KB, text/plain)
2003-04-14 13:34 UTC, Jiri Kovalsky
Details
A new binary patch. Put into <NB-install>/modules/autoload/patches/org-netbeans-modules-vcscore/ folder. (10.03 KB, application/octet-stream)
2003-04-14 13:40 UTC, Martin Entlicher
Details
Attaching thread dump while invocation of the wizard ... (12.82 KB, text/plain)
2003-04-14 13:52 UTC, Jiri Kovalsky
Details
A new binary patch. Put into <NB-install>/modules/autoload/patches/org-netbeans-modules-vcscore/ folder. (10.04 KB, application/octet-stream)
2003-04-14 15:39 UTC, Martin Entlicher
Details
The new contextual diff of the fix. (7.96 KB, patch)
2003-04-14 16:25 UTC, Martin Entlicher
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jiri Kovalsky 2003-04-11 11:00:04 UTC
Development build #200304102350 of NetBeans 3.5
Windows 2000 with JDK 1.4.1 build #21

Description:
============
It takes pretty long time to display mounting
wizard after it is invoked from menu. Quickly
measured it is about 8 seconds ! I am attaching
set of thread dumps taken during this period. IDE
takes nearly 100% CPU and is irresponsive.

Steps to reproduce:
===================
1. Just invoke "Versioning|Mount Version
Control|Generic VCS" item from main menu.
Comment 1 Jiri Kovalsky 2003-04-11 11:00:52 UTC
This is serious preformance regression.
Comment 2 Jiri Kovalsky 2003-04-11 11:44:11 UTC
Created attachment 9870 [details]
Several thread dumps taken during wizard invocation.
Comment 3 Martin Entlicher 2003-04-14 12:14:52 UTC
Fixed in the main trunk, the object integrity support is initialized
lazily not to block the filesystem instantiation:

/cvs/vcscore/src/org/netbeans/modules/vcscore/objectintegrity/IntegritySupportMaintainer.java,v 
<--  IntegritySupportMaintainer.java
new revision: 1.3; previous revision: 1.2
/cvs/vcscore/src/org/netbeans/modules/vcscore/objectintegrity/VcsObjectIntegritySupport.java,v 
<--  VcsObjectIntegritySupport.java
new revision: 1.11; previous revision: 1.10
Comment 4 Martin Entlicher 2003-04-14 12:17:08 UTC
Created attachment 9904 [details]
The binary patch. Put into <NB-install>/modules/autoload/patches/org-netbeans-modules-vcscore/ folder.
Comment 5 Martin Entlicher 2003-04-14 12:21:03 UTC
Created attachment 9905 [details]
The contextual diff, that fix this problem.
Comment 6 Jiri Kovalsky 2003-04-14 13:14:55 UTC
I am sorry but this does not work. I got two exceptions and lost my
filesystems.
Comment 7 Jiri Kovalsky 2003-04-14 13:16:23 UTC
Created attachment 9908 [details]
Here are those exceptions ...
Comment 8 Jiri Kovalsky 2003-04-14 13:34:01 UTC
Created attachment 9910 [details]
Hm, probably this is what I should attach, I apologize ... :-(
Comment 9 Martin Entlicher 2003-04-14 13:38:48 UTC
Well, the first patch assumed some other patch which is already in the
main trunk. I'm going to attach a new one, that should work with
current NB 3.5 / S1S 5.0.
Comment 10 Martin Entlicher 2003-04-14 13:40:14 UTC
Created attachment 9911 [details]
A new binary patch. Put into <NB-install>/modules/autoload/patches/org-netbeans-modules-vcscore/ folder.
Comment 11 Jiri Kovalsky 2003-04-14 13:51:20 UTC
Sorry, nothing has changed i.e. still takes looong time. :-)
Comment 12 Jiri Kovalsky 2003-04-14 13:52:39 UTC
Created attachment 9912 [details]
Attaching thread dump while invocation of the wizard ...
Comment 13 Martin Entlicher 2003-04-14 14:40:54 UTC
Hmm, there is one more thing, that triggered the loading process.
I'm going to improve the fix...
 
Comment 14 Martin Entlicher 2003-04-14 15:36:43 UTC
So the initialization is done even more lazily. Fixed in the main
trunk:

/cvs/vcscore/src/org/netbeans/modules/vcscore/objectintegrity/VcsObjectIntegritySupport.java,v 
<--  VcsObjectIntegritySupport.java
new revision: 1.12; previous revision: 1.11
Comment 15 Martin Entlicher 2003-04-14 15:39:36 UTC
Created attachment 9920 [details]
A new binary patch. Put into <NB-install>/modules/autoload/patches/org-netbeans-modules-vcscore/ folder.
Comment 16 Jiri Kovalsky 2003-04-14 15:57:01 UTC
Hm, this is great. I haven't noticed any performance delay at all !
Where did you Martin hide it ? Verified with the last version of the
patch in development build #200304132350 of NetBeans 3.5.
Comment 17 Martin Entlicher 2003-04-14 16:25:39 UTC
Created attachment 9923 [details]
The new contextual diff of the fix.
Comment 18 Richard Gregor 2003-04-14 16:40:06 UTC
Code reviewed without objections.
Comment 19 _ ttran 2003-04-14 16:52:24 UTC
approved for 3.5
Comment 20 Martin Entlicher 2003-04-14 16:59:49 UTC
Thanks for the review, verification and approval. The problem is fixed
in release35 branch:

Checking in
org/netbeans/modules/vcscore/objectintegrity/IntegritySupportMaintainer.java;
/shared/data/helm/cvs/repository/vcscore/src/org/netbeans/modules/vcscore/objectintegrity/IntegritySupportMaintainer.java,v 
<--  IntegritySupportMaintainer.java
new revision: 1.1.2.3; previous revision: 1.1.2.2
done
Checking in
org/netbeans/modules/vcscore/objectintegrity/VcsObjectIntegritySupport.java;
/shared/data/helm/cvs/repository/vcscore/src/org/netbeans/modules/vcscore/objectintegrity/VcsObjectIntegritySupport.java,v 
<--  VcsObjectIntegritySupport.java
new revision: 1.2.2.6; previous revision: 1.2.2.5
done
Comment 21 Jiri Kovalsky 2003-07-08 13:04:43 UTC
Great, no performance degradation. Verified in Sun ONE Studio 5.0
Standard Edition build #030528.