Bug 33363 - Invalid use of Thread.sleep in DeployProgressMonitor
Invalid use of Thread.sleep in DeployProgressMonitor
Status: RESOLVED FIXED
Product: serverplugins
Classification: Unclassified
Component: Infrastructure
3.x
PC Linux
: P2 (vote)
: 4.x
Assigned To: _ gfink
issues@serverplugins
: PERFORMANCE, THREAD
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2003-05-02 13:17 UTC by _ rkubacki
Modified: 2006-06-05 00:39 UTC (History)
0 users

See Also:
Issue Type: DEFECT
:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description _ rkubacki 2003-05-02 13:17:41 UTC
Event listeners must not needlessly delay program
execution. Use of Thread.sleep(400) in
handleProgressEvent is completely wrong as it will
block caller (that might be communicating with
remote client so any delay can be source of troubles).

Also if I use apparent workaround and fire event
asynchronously from newly created Thread or just
by calling RequestProcessor.getDefault.post() it
will not achieve its purpose at all.
Comment 1 Nam Nguyen 2004-01-20 21:56:33 UTC
Remove sleep(400) in handleProgressEvent.


By use of this website, you agree to the NetBeans Policies and Terms of Use. © 2012, Oracle Corporation and/or its affiliates. Sponsored by Oracle logo