[Checkins] SVN: Sandbox/adamg/ocql/branches/optimize-with-index/src/ocql/database/metadata.py don't use test class properties here

Adam Groszer agroszer at gmail.com
Sun Jul 13 05:37:07 EDT 2008


Log message for revision 88309:
  don't use test class properties here
  use AttributeIndex instead of FieldIndex

Changed:
  U   Sandbox/adamg/ocql/branches/optimize-with-index/src/ocql/database/metadata.py

-=-
Modified: Sandbox/adamg/ocql/branches/optimize-with-index/src/ocql/database/metadata.py
===================================================================
--- Sandbox/adamg/ocql/branches/optimize-with-index/src/ocql/database/metadata.py	2008-07-12 23:36:10 UTC (rev 88308)
+++ Sandbox/adamg/ocql/branches/optimize-with-index/src/ocql/database/metadata.py	2008-07-13 09:37:05 UTC (rev 88309)
@@ -5,7 +5,7 @@
 from zope.component import getUtility
 from zope.component import getUtilitiesFor
 from zope.app.catalog.interfaces import ICatalog
-from zope.app.catalog.field import FieldIndex
+from zope.app.catalog.attribute import AttributeIndex
 from zope.app.intid import IIntIds
 import zc.relation.interfaces
 
@@ -100,25 +100,24 @@
         intids = getUtility(IIntIds)
         for name, catalog in catalogs:
             for iname, index in catalog.items():
-                if isinstance(index, FieldIndex) and \
+                if isinstance(index, AttributeIndex) and \
                 index.field_name == property and \
                 index.interface.__name__ == klass:
                     results = catalog.apply({iname:(lowerbound, upperbound)})
-                    obj_list = [intids.getObject(result).name for result in results]
+                    obj_list = [intids.getObject(result) for result in results]
                     return obj_list
 
     def hasPropertyIndex(self, klass, property):
         catalogs = getUtilitiesFor(ICatalog)
-        intids = getUtility(IIntIds)
         for name, catalog in catalogs:
             for iname, index in catalog.items():
-                if isinstance(index, FieldIndex) and \
+                if isinstance(index, AttributeIndex) and \
                 index.field_name == property and \
                 index.interface.__name__ == klass:
                     return True
         return False
-          
 
+
     def get_class(self, klassname):
         """Returns a MetaType instance for the class."""
         return self.classes[klassname]



More information about the Checkins mailing list