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 106068 - PlatformCatalogAutoInstaller & MidpValidatorWarmUp should not be in WarmUp
Summary: PlatformCatalogAutoInstaller & MidpValidatorWarmUp should not be in WarmUp
Status: NEW
Alias: None
Product: javame
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: Roman Svitanic
URL:
Keywords: PERFORMANCE
: 108401 (view as bug list)
Depends on: 106611
Blocks:
  Show dependency tree
 
Reported: 2007-06-07 22:43 UTC by Jesse Glick
Modified: 2013-04-09 11:22 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 Jesse Glick 2007-06-07 22:43:18 UTC
If you mean to use ModuleInstall, use it. Otherwise this instance should very
likely be removed from WarmUp.
Comment 1 Adam Sotona 2007-06-08 09:23:49 UTC
Why?
Starting heavyweight platform installation directly from module install (even in
a separate thread) would be serious performace issue for IDE startup.
Comment 2 Petr Nejedly 2007-06-08 14:15:56 UTC
We're glad that you don't slow down the startup by the heavyweight platform
installation, but WarmUp is not good place for this.
The semantic of warmup folder is: Put there something that can preinitialize
things, but not something that is necessary to be called.
If the IDE decides to skip warmup, it must not affect functionality of your code.

Is it clear now? 
Comment 3 Adam Sotona 2007-06-14 09:34:03 UTC
The implementation of platform installation was changed after a suggestion from performance team some time ago.
The semantics of warmup probably changed so please provide a non-optional warmaup to be able to fix this issue (filled
issue #106611).
Comment 4 Adam Sotona 2007-07-31 15:18:56 UTC
*** Issue 108401 has been marked as a duplicate of this issue. ***
Comment 5 _ tboudreau 2008-10-02 19:53:30 UTC
Isn't there some way to run this code, say from a static block the first time something mobility-related is touched?
Comment 6 Jesse Glick 2009-09-22 16:52:38 UTC
Issue #108401 mentions KeyStoreRepositoryWarmUp but I don't see any such class in the repo any more. I do see
MidpValidatorWarmUp in the WarmUp folder in

http://deadlock.netbeans.org/hudson/job/nbms-and-javadoc/lastSuccessfulBuild/artifact/nbbuild/build/generated/layers.txt

Please reevaluate:

1. If these tasks _must_ be executed during startup, before startup completes, then ModuleInstall.restored() is appropriate.

2. If these tasks _must_ be executed soon after startup, you can spawn a RequestProcessor task from ModuleInstall with a
lower priority, perhaps with a delay.

3. If these tasks _may_ be executed to improve responsiveness later, but have no other user-visible effect, then WarmUp
would be appropriate.

4. More likely, these tasks need not be executed during startup at all. Perhaps they could be run if and when the first
mobility project is opened (which might happen to be during startup in case the IDE was shut down with such a project
open). This is far better since it avoids doing any work when Mobility features are not actually used.
Comment 7 Petr Suchomel 2009-09-22 21:37:39 UTC
KeyStoreRepositoryWarmUp was solved, MidpValidatorWarmUp was some VMD performance workaround - Tim B. made this. There
is no escape from PlatformCatalogAutoInstaller - this should be moved to ModuleInstall.restored() then.
Comment 8 David Strupl 2013-04-09 11:22:58 UTC
Assigning to the new owner of javame modules.