Bug 322261 - WTP needs an API on ServerPublishInfo to rebuild the cache for a specific module
Summary: WTP needs an API on ServerPublishInfo to rebuild the cache for a specific module
Status: RESOLVED FIXED
Alias: None
Product: WTP ServerTools
Classification: WebTools
Component: wst.server (show other bugs)
Version: 3.0.5   Edit
Hardware: PC Windows XP
: P1 normal (vote)
Target Milestone: 3.0.5 P   Edit
Assignee: Angel Vera CLA
QA Contact: Angel Vera CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-10 13:49 EDT by Troy Bishop CLA
Modified: 2010-08-13 17:27 EDT (History)
1 user (show)

See Also:


Attachments
possible patch to allow adaptors to destroy/rebuild the cache of a specific IModule (1.24 KB, patch)
2010-08-10 13:50 EDT, Troy Bishop CLA
arvera: iplog+
arvera: review+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Troy Bishop CLA 2010-08-10 13:49:36 EDT
Build Identifier: 3.0.5_patches

Within a product that adopts WTP we are experiencing a rather bad publishing performance problem.  This is a result of our product executing code generators as part of the publishers framework and our publisher destroying the entire WTP delta cache when those code generators complete.  We have to destroy the entire publishing cache because there is no finer-grained API to only destroy the modules that were affected by these code generators.

The attached patch will allow us to selectively destroy the cache so that we are not completely invalidating the entire cache.

Reproducible: Always
Comment 1 Troy Bishop CLA 2010-08-10 13:50:29 EDT
Created attachment 176258 [details]
possible patch to allow adaptors to destroy/rebuild the cache of a specific IModule
Comment 2 Troy Bishop CLA 2010-08-10 13:55:04 EDT
Adding hot bug request for including this patch into 305_patches:

1. Affiliation.

IBM

2. Be clear on which release you want this bug to be fixed in, or "last workable" date, if requesting a patch.

3.0.5 patches

3. Justify why this is a hotbug. Note that "our users don't like the bug" is not the type of reason that gets much attention. The motivation we are looking for to justify a P1-like priority is, basically, "this bug blocks our adoption of WTP" (explaining why, of course).

Introducing this API will allow our WTP adaptor product to selectively destroy the WTP publishing IModule cache when it executes its code-generators added as part of the publishers framework.  Without this change we have to destroy the entire cache which means that the publish will need to perform an entire application delta computation for a second time and this can (when the application is very large) introduce significant publishing performance problems.
Comment 3 Troy Bishop CLA 2010-08-10 13:56:52 EDT
One additional comment: this change does not need to be put into HEAD as the rebuildCache() API already exists there (from bug Bug 319288).
Comment 4 Carl Anderson CLA 2010-08-13 17:27:58 EDT
Commited to R3_0_5_patches