[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