[Neo-report] r2098 gregory - in /trunk/neo: master/handlers/storage.py storage/app.py

[nobody at svn.erp5.org]

Author: gregory
Date: Thu May 13 14:02:24 2010
New Revision: 2098

Log:
Select only identified client nodes.

Between closure request and effect, a client node can be registered to the
node manager without a connection. As there is no needs to use an
unconnected client node, always set only_identified parameter to True.

Modified:
    trunk/neo/master/handlers/storage.py
    trunk/neo/storage/app.py

Modified: trunk/neo/master/handlers/storage.py
==============================================================================
--- trunk/neo/master/handlers/storage.py [iso-8859-1] (original)
+++ trunk/neo/master/handlers/storage.py [iso-8859-1] Thu May 13 14:02:24 2010
@@ -73,7 +73,7 @@
         transaction_node = t.getNode()
         invalidate_objects = Packets.InvalidateObjects(t.getOIDList(), tid)
         answer_transaction_finished = Packets.AnswerTransactionFinished(tid)
-        for client_node in nm.getClientList():
+        for client_node in nm.getClientList(only_identified=True):
             c = client_node.getConnection()
             if client_node is transaction_node:
                 c.answer(answer_transaction_finished, msg_id=t.getMessageId())

Modified: trunk/neo/storage/app.py
==============================================================================
--- trunk/neo/storage/app.py [iso-8859-1] (original)
+++ trunk/neo/storage/app.py [iso-8859-1] Thu May 13 14:02:24 2010
@@ -167,7 +167,7 @@
             if node is not None and node.isHidden():
                 self.wait()
             # drop any client node and clear event queue
-            for node in self.nm.getClientList():
+            for node in self.nm.getClientList(only_identified=True):
                 node.getConnection().close()
             self.event_queue = deque()
             try: