[Grok-dev] Re: catalog indexes in Grok

[Martijn Faassen]

Hey,

Philipp von Weitershausen wrote:
> I played with the faassen-index branch a bit just now. I ported 
> NudgeNudge to use it: 
> http://codespeak.net/svn/z3/NudgeNudge/branches/indexes-refactoring

Cool! Thanks for trying this out. I hope to merge this stuff by 
tomorrow. I need to do some more work first: make it work for sites, not 
just for applications, and extending the test coverage and error 
handling somewhat.

Is a merge okay from your perspective?

> I was using zc.catalog's indexes (namely ValueIndex) in NudgeNudge 
> before. Are there any reasons why we wouldn't want to make Grok 
> dependent on zc.catalog and make its indexes available as declarations 
> from grok.index?

No reasons not to that I'm aware of. I just hadn't gotten around to it yet.

Regards,

Martijn