Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

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

(In [7910]) Fixing dangling clients which close import sessions (See #2804, Fix #2772, #2803)

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 [8262]) Copying reference counts between SessionContexts (See #2804, Fix #2803)

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.72136 sec.)

We're Hiring!