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 146225 - [65cat] AssertionError at org.netbeans.modules.db.explorer.ConnectionList.remove
Summary: [65cat] AssertionError at org.netbeans.modules.db.explorer.ConnectionList.remove
Status: VERIFIED FIXED
Alias: None
Product: db
Classification: Unclassified
Component: Code (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: Rob Englander
URL: http://statistics.netbeans.org/except...
Keywords:
Depends on:
Blocks:
 
Reported: 2008-09-04 17:30 UTC by gholmer
Modified: 2008-09-22 10:50 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter: 87936


Attachments
stacktrace (1.35 KB, text/plain)
2008-09-04 17:30 UTC, gholmer
Details
stacktrace (25 bytes, text/plain)
2008-09-06 10:55 UTC, qingyue
Details
stacktrace (1.35 KB, text/plain)
2008-09-06 10:55 UTC, qingyue
Details

Note You need to log in before you can comment on or make changes to this bug.
Description gholmer 2008-09-04 17:30:28 UTC
Build: NetBeans IDE Dev (Build 200809020201)
VM: Java HotSpot(TM) Client VM, 11.0-b15, Java(TM) SE Runtime Environment, 1.6.0_10-rc-b28
OS: Linux, 2.6.24-19-generic, i386

User Comments: 
Delete existing database connection (Services tab, Databases node).

Stacktrace: 
java.lang.AssertionError
        at org.netbeans.modules.db.explorer.ConnectionList.remove(ConnectionList.java:164)
        at org.netbeans.modules.db.explorer.infos.ConnectionNodeInfo.delete(ConnectionNodeInfo.java:462)
        at org.netbeans.modules.db.explorer.nodes.DatabaseNode.destroy(DatabaseNode.java:201)
        at org.openide.nodes.FilterNode.destroy(FilterNode.java:599)
        at org.openide.explorer.ExplorerActionsImpl$DeleteActionPerformer.doDestroy(ExplorerActionsImpl.java:640)
        at org.openide.explorer.ExplorerActionsImpl$DeleteActionPerformer.actionPerformed(ExplorerActionsImpl.java:590)
Comment 1 gholmer 2008-09-04 17:30:34 UTC
Created attachment 69085 [details]
stacktrace
Comment 2 qingyue 2008-09-06 10:55:38 UTC
Build: NetBeans IDE Dev (Build 200809040201)
VM: Java HotSpot(TM) Client VM, 11.0-b12, Java(TM) SE Runtime Environment, 1.6.0_10-beta-b24
OS: Windows XP, 5.1, x86

User Comments: 


Stacktrace: 
java.lang.AssertionError
        at org.netbeans.modules.db.explorer.ConnectionList.remove(ConnectionList.java:164)
        at org.netbeans.modules.db.explorer.infos.ConnectionNodeInfo.delete(ConnectionNodeInfo.java:462)
        at org.netbeans.modules.db.explorer.nodes.DatabaseNode.destroy(DatabaseNode.java:201)
        at org.openide.nodes.FilterNode.destroy(FilterNode.java:599)
        at org.openide.explorer.ExplorerActionsImpl$DeleteActionPerformer.doDestroy(ExplorerActionsImpl.java:640)
        at org.openide.explorer.ExplorerActionsImpl$DeleteActionPerformer.actionPerformed(ExplorerActionsImpl.java:590)
Comment 3 qingyue 2008-09-06 10:55:39 UTC
Build: NetBeans IDE Dev (Build 200809040201)
VM: Java HotSpot(TM) Client VM, 11.0-b12, Java(TM) SE Runtime Environment, 1.6.0_10-beta-b24
OS: Windows XP, 5.1, x86

User Comments: 


Stacktrace: 
java.lang.AssertionError
Comment 4 qingyue 2008-09-06 10:55:42 UTC
Created attachment 69215 [details]
stacktrace
Comment 5 David Vancouvering 2008-09-09 01:15:50 UTC
Can't reproduce, of course.  It's a timing bug and now I have to sit and think and analyze and try and figure out how
this happens...  Sigh...
Comment 6 David Vancouvering 2008-09-09 01:17:16 UTC
OK, I looked at the code some more.  This happens when we try to remove a connection from our internal connection list
and the connection is not there.

Actually, that's probably OK, and is a sign of a slightly over-conservative assert.  I'll change it to an informational
log message just so we know that something's awry but go ahead and let the operation continue.
Comment 7 David Vancouvering 2008-09-09 01:20:22 UTC
19651e1cea5a
Comment 8 Andrei Badea 2008-09-09 10:50:53 UTC
I will note the that the fix is probably working around another problem, so I do not agree with it. (Assuming the fix
removes the assert, as stated in desc7.)
Comment 9 Quality Engineering 2008-09-10 05:44:47 UTC
Integrated into 'main-golden', will be available in build *200809100201* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/19651e1cea5a
User: David Van Couvering <davidvc@netbeans.org>
Log: #146225: [65cat] AssertionError at org.netbeans.modules.db.explorer.ConnectionList.remove
Comment 10 Roman Mostyka 2008-09-11 15:58:49 UTC
gholmer, can you verify this issue to be sure that it is fixed?
David, what's about Andrei's comment in desc9?
Comment 11 David Vancouvering 2008-09-11 16:47:33 UTC
Andrei has a good point that an assert is there for a reason and we should try and identify the root cause.  So we can
keep this open, but at least people won't keep getting assertion errors when the system is still functioning fine.
Comment 12 gholmer 2008-09-11 17:13:16 UTC
> gholmer, can you verify this issue to be sure that it is fixed?

I don't see it happening any more, thanks.
Comment 13 David Vancouvering 2008-09-16 21:23:05 UTC
Hi, Rob.  This is related to the connection cache.  I suspect if you remove the cache, then is issue becomes a
non-issue.  Note that right now the assertion has been removed, but there are cases when we are asked to remove a
connection that is not in the cache.  Probably related to the work you are doing with ConnectionNodeInfo.
Comment 14 Rob Englander 2008-09-19 18:35:05 UTC
Fixed in #4c829f9abb13 for Issue 144819.
Comment 15 Roman Mostyka 2008-09-22 10:50:32 UTC
Verified with build 080921.