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

Summary: Looong time to display Generic VCS mounting wizard.
Product: obsolete Reporter: Jiri Kovalsky <jkovalsky>
Component: vcscoreAssignee: Martin Entlicher <mentlicher>
Status: VERIFIED FIXED    
Severity: blocker Keywords: PERFORMANCE
Priority: P2    
Version: -S1S-   
Hardware: PC   
OS: Windows ME/2000   
Issue Type: DEFECT Exception Reporter:
Attachments: Several thread dumps taken during wizard invocation.
The binary patch. Put into <NB-install>/modules/autoload/patches/org-netbeans-modules-vcscore/ folder.
The contextual diff, that fix this problem.
Here are those exceptions ...
Hm, probably this is what I should attach, I apologize ... :-(
A new binary patch. Put into <NB-install>/modules/autoload/patches/org-netbeans-modules-vcscore/ folder.
Attaching thread dump while invocation of the wizard ...
A new binary patch. Put into <NB-install>/modules/autoload/patches/org-netbeans-modules-vcscore/ folder.
The new contextual diff of the fix.

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.