Context Navigation

← Previous Ticket
Next Ticket →

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 #3631 (closed)

Opened 13 years ago

Closed 13 years ago

Bug: new session code doesn't timeout sessions

Reported by:	jamoore	Owned by:	jamoore
Priority:	blocker	Milestone:	OMERO-Beta4.3
Component:	General	Version:	n.a.
Keywords:	n.a.	Cc:	cxallan, cneves
Resources:	n.a.	Referenced By:	n.a.
References:	n.a.	Remaining Time:	n.a.
Sprint:	2011-01-13 (3)

Description

After days of not logging into to a server, I tried the following:

~/code/git/dist $ bin/omero hql -q "from Dataset"
Using session 1bc1094d-8365-4c56-9b76-eed79a9a8ba3 (root@178.63.69.8). Idle timeout: 10.0 min. Current group: system
 # 
---
(0 rows)
~/code/git/dis

Looking on the server for that session:

2010-12-03 10:26:51,610 INFO  [        ome.services.util.ServiceHandler] (3-thread-5)  Executor.doWork -- ome.services.sessions.SessionManagerImpl.reload(1bc1094d-8365-4c56-9b76-eed79a9a8ba3)
...
2010-12-03 12:03:10,579 INFO  [e.services.sessions.SessionContext$Count] (l.Server-2) -Reference count: 1bc1094d-8365-4c56-9b76-eed79a9a8ba3=0
2010-12-03 12:04:51,605 INFO  [        ome.services.util.ServiceHandler] (3-thread-4)  Executor.doWork -- ome.services.sessions.SessionManagerImpl.reload(1bc1094d-8365-4c56-9b76-eed79a9a8ba3)
2010-12-03 12:06:51,606 INFO  [        ome.services.util.ServiceHandler] (3-thread-3)  Executor.doWork -- ome.services.sessions.SessionManagerImpl.reload(1bc1094d-8365-4c56-9b76-eed79a9a8ba3)

i.e. the session cache is examining the session during each reload, but not getting rid of it despite the fact that it should have timed out.

References

Referenced by:
← User Story (#3173): Increase session store concurrency

Change History (6)

comment:1 Changed 13 years ago by jmoore

Owner set to jmoore

comment:2 Changed 13 years ago by jburel

Sprint changed from 2010-12-09 (21) to 2010-12-23 (22)

Moved from sprint 2010-12-09 (21)

comment:3 Changed 13 years ago by jmoore

Issue solved in local repository. Was an issue of incorrectly upping the lastAccess stamp on reload(). There was another issue with concurrency (#3664) which will be handled separately. (Much lower priority)

comment:4 Changed 13 years ago by cneves

(In [8743]) Josh's Fix for session leak from r8450 (See #3631)

comment:5 Changed 13 years ago by jburel

Sprint changed from 2010-12-23 (22) to 2011-01-13 (23)

Moved from sprint 2010-12-23 (22)

comment:6 Changed 13 years ago by jmoore

Resolution set to fixed
Status changed from new to closed

Fix from branch merged into r8775

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

Download in other formats: