[Erp5-report] r27219 - /erp5/trunk/products/ERP5Type/Tool/WebServiceTool.py

[nobody at svn.erp5.org]

Author: vincent
Date: Wed May 27 17:59:19 2009
New Revision: 27219

URL: http://svn.erp5.org?rev=27219&view=rev
Log:
Caching cannot and must not be done at this level, because it can be desired to get different connections.
As an example, in MySQL transaction information is local to a connection. So returning an existing connection is not always the same as creating a new one, and only the class doing the actual connection knows what can be shared and what cannot.

Modified:
    erp5/trunk/products/ERP5Type/Tool/WebServiceTool.py

Modified: erp5/trunk/products/ERP5Type/Tool/WebServiceTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Tool/WebServiceTool.py?rev=27219&r1=27218&r2=27219&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Tool/WebServiceTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/Tool/WebServiceTool.py [utf8] Wed May 27 17:59:19 2009
@@ -87,7 +87,6 @@
     of any kind of web service (not only ERP5) with many
     different kinds of transport like 'xml-rpc' or 'soap'
     """
-    # XXX: implement connection caching per zope thread
     if transport_kw is None:
       transport_kw = {}
     connection_handler_klass = connection_plugin_registry[transport]