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 85995 - [55cat] Deadlock with NNMDRListener and CVS operation
Summary: [55cat] Deadlock with NNMDRListener and CVS operation
Status: RESOLVED FIXED
Alias: None
Product: javaee
Classification: Unclassified
Component: Code (show other bugs)
Version: 5.x
Hardware: PC Windows XP
: P2 blocker (vote)
Assignee: Andrei Badea
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-09-28 06:52 UTC by wulgar
Modified: 2006-10-10 17:47 UTC (History)
6 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
thread dump (65.94 KB, text/plain)
2006-09-28 06:53 UTC, wulgar 		Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description wulgar 2006-09-28 06:52:48 UTC 
[ BUILD # : 200609270000 ]
[ JDK VERSION : 1.6 ]

When I try to update files from CVS deadlock occured. See Attachment.
Comment 1 wulgar 2006-09-28 06:53:19 UTC 
Created attachment 34650 [details]
thread dump
Comment 2 Tomas Zezula 2006-10-02 12:11:47 UTC 
Caused by URLMapper.findFileObject which fires events.
Comment 3 rmatous 2006-10-02 12:48:03 UTC 
Hardly to blame filesystems in this case. Its disputable whether FileObject may
fire events synchronously from getFileObject. But definitely don't expect any
changes in filesystems to release55 because its very risky. If you need to
postpone firing events from filesystems, please use AtomicAction.
Comment 4 Tomas Zezula 2006-10-02 12:54:43 UTC 
Radek's solution (AtomicAction) is probably the best one for 5.5. To solve the
problem in the classpath, it would require to call FS outside synchronized block
(split of sync block) which may be very dangerous. The best fix is to use
AtomicAction in the
org.netbeans.modules.j2ee.metadata.ClassPathSourceCache.getSourceRootSet.
Comment 5 Andrei Badea 2006-10-02 13:40:20 UTC 
Taking over then.
Comment 7 Martin Schovanek 2006-10-03 14:27:44 UTC 
verified in release55_dev
Comment 8 rmatous 2006-10-10 17:22:47 UTC 
Seems OK for me.