Changeset 2592 for trunk/components/blitz/src/ome/services/blitz/impl/ServiceFactoryI.java

Timestamp:

07/05/08 22:16:04 (5 months ago)

Author:

jmoore

Message:

ticket:1018 - Initial version of OmeroSessions reference counting

Implementation is incomplete in the blitz case since there are two
possible "closers" - the user and the glacier session, which must
be taken into account.

Files:

• trunk/components/blitz/src/ome/services/blitz/impl/ServiceFactoryI.java (modified) (1 diff)

trunk/components/blitz/src/ome/services/blitz/impl/ServiceFactoryI.java

@@ -524 +524 @@
     }
 
+    /**
+     * Destruction simply decrements the reference count for a session to allow
+     * reconnecting to it. This means that the Glacier timeout property is
+     * fairly unimportant. If a Glacier connection times out or is otherwise
+     * destroyed, a client can attempt to reconnect
+     */
     public void destroy(Ice.Current current) {
-        if (log.isInfoEnabled()) {
-            log.info(String.format("Destroying %s session", current.id.name));
-        }
-        close(current);
+        sessionManager.detach(this.principal.getName());
     }