Task #2803 (closed)
Opened 14 years ago
Closed 13 years ago
BUG: session reference count not copied on update
Reported by: | jamoore | Owned by: | jamoore |
---|---|---|---|
Priority: | blocker | Milestone: | OMERO-Beta4.2.1 |
Component: | Import | Version: | n.a. |
Keywords: | n.a. | Cc: | cxallan, dzmacdonald |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | 0.0d |
Sprint: | 2010-09-30 (17) |
Description (last modified by jmoore)
In several places (e.g. SessionManagerImpl update() and reload()), SessionContextImpl instances are recreated with the values from previous contexts without having the reference values copied leading to exceptions like the one below. (It's unlikely that the previous commits to fix this had any lasting effect).
This issue was discovered via the many logging statements added as commits to #2804, which is partially but not completely related.
Previous description
During really long method calls (saveToDb) sessions are being closed by some combination of left over sessions. (#2772 and from OmeroCli).
Caused by: omero.RemovedSessionException serverStackTrace = "ome.conditions.RemovedSessionException: No context for d17aea6c-ba7c-4864-a9b5-35227b430446 at ome.services.sessions.state.SessionCache.getElementNullOrThrowOnTimeout(SessionCache.java:269) at ome.services.sessions.state.SessionCache.getSessionContext(SessionCache.java:235) at ome.services.sessions.SessionManagerImpl.getSessionStats(SessionManagerImpl.java:515) at ome.services.sessions.stats.PerSessionStats.stats(PerSessionStats.java:31) at ome.services.sessions.stats.DelegatingStats.updatedObjects(DelegatingStats.java:75)
Change History (7)
comment:1 Changed 14 years ago by jmoore
- Owner set to jmoore
- Status changed from new to assigned
comment:2 Changed 14 years ago by jmoore
- Remaining Time changed from 0.25 to 0
- Resolution set to fixed
- Status changed from assigned to closed
comment:3 Changed 13 years ago by jmoore
- Description modified (diff)
- Remaining Time changed from 0 to 0.25
- Resolution fixed deleted
- Sprint changed from 2010-08-26 (15) to 2010-09-30 (17)
- Status changed from closed to reopened
- Summary changed from Import fails after 10 minutes with "session removed" to BUG: session reference count not copied on update
comment:4 Changed 13 years ago by jmoore
- Description modified (diff)
comment:5 Changed 13 years ago by jmoore
- Status changed from reopened to new
comment:6 Changed 13 years ago by jmoore
- Status changed from new to assigned
comment:7 Changed 13 years ago by jmoore
- Remaining Time changed from 0.25 to 0
- Resolution set to fixed
- Status changed from assigned to closed
(In [7910]) Fixing dangling clients which close import sessions (See #2804, Fix #2772, #2803)