Bug 105163 - [perf] propagation of corexception in publishModule slow
Summary: [perf] propagation of corexception in publishModule slow
Status: CLOSED WORKSFORME
Alias: None
Product: WTP ServerTools
Classification: WebTools
Component: wst.server (show other bugs)
Version: 1.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Tim deBoer CLA
QA Contact:
URL:
Whiteboard:
Keywords: performance
Depends on:
Blocks:
 
Reported: 2005-07-26 11:07 EDT by Sachin Patel CLA
Modified: 2017-10-11 15:49 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sachin Patel CLA 2005-07-26 11:07:05 EDT
Whenever an error occurs in the implementation of 

public void publishModule(int kind, int deltaKind, IModule[] module,
IProgressMonitor monitor)
			throws CoreException 

for a given ServerBehavior an a corexception is delegated back up, even through
exited there seems to be a hang for about 5-10 seconds before the error is
displayed to the user.  This should be investigated, as during this 5-10 seconds
the UI is locked.

If no exception occurs, then the publish job ends quickly.
Comment 1 Tim deBoer CLA 2005-08-18 10:01:41 EDT
Hi Sachin,

I couldn't reproduce this - a CoreException thrown within a server behaviour
delegate was displayed quickly and with no UI lockup. It does finish the
publishing job normally (i.e. publish other modules and then close the
connection to the server) before returning the error, so my only guess would be
that the server type you were dealing with took time to publish other modules,
although that doesn't explain the UI lockup.

Since I can't reproduce I'm returning this bug. If you still see this and can
provide more information please feel free to reopen.
Comment 2 Tim deBoer CLA 2005-12-13 10:25:41 EST
Closing old bugs that haven't had any activity in a long time.
Comment 3 David Williams CLA 2006-09-07 09:30:33 EDT
performance bugs should use performance keyword (not [performance] in subject). 
Comment 4 Eclipse Genie CLA 2017-10-11 15:49:35 EDT
New Gerrit change created: https://git.eclipse.org/r/107353