[Erp5-report] r17041 - /erp5/trunk/products/ZSQLCatalog/SQLCatalog.py

[nobody at svn.erp5.org]

Author: vincent
Date: Thu Oct 18 18:31:52 2007
New Revision: 17041

URL: http://svn.erp5.org?rev=17041&view=rev
Log:
Warn about bad volatile usage.

Modified:
    erp5/trunk/products/ZSQLCatalog/SQLCatalog.py

Modified: erp5/trunk/products/ZSQLCatalog/SQLCatalog.py
URL: http://svn.erp5.org/erp5/trunk/products/ZSQLCatalog/SQLCatalog.py?rev=17041&r1=17040&r2=17041&view=diff
==============================================================================
--- erp5/trunk/products/ZSQLCatalog/SQLCatalog.py (original)
+++ erp5/trunk/products/ZSQLCatalog/SQLCatalog.py Thu Oct 18 18:31:52 2007
@@ -1123,6 +1123,9 @@
     return self.sql_search_result_keys
 
   def _getCatalogSchema(self, table=None):
+    # XXX: Using a volatile as a cache makes it impossible to flush
+    # consistently on all connections containing the volatile. Another
+    # caching scheme must be used here.
     catalog_schema_dict = getattr(aq_base(self), '_v_catalog_schema_dict', {})
 
     if table not in catalog_schema_dict: