lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Erik Hatcher (JIRA)" <>
Subject [jira] Commented: (LUCENE-1387) Add LocalLucene
Date Fri, 19 Dec 2008 11:07:44 GMT


Erik Hatcher commented on LUCENE-1387:

I've taken some quick peeks into the code, run the unit tests, nicely packaged and presented!

A couple of thoughts:

* Maybe the Filter's should be using the DocIdSet API rather than the BitSet deprecated stuff?
 We can refactor that after being committed I supposed, but not something we want to leave
like that.

* DistanceQuery is awkwardly named.  It's not an (extends) Query.... it's a POJO with helpers.
 Maybe DistanceQueryFactory?   (but it creates a Filter also)

* CartesianPolyFilter is not a Filter (but CartesianShapeFilter is)

I think this looks good enough to commit as well, just noting the above for cosmetic refactoring
consideration after the code is in.

> Add LocalLucene
> ---------------
>                 Key: LUCENE-1387
>                 URL:
>             Project: Lucene - Java
>          Issue Type: New Feature
>          Components: contrib/*
>            Reporter: Grant Ingersoll
>            Priority: Minor
>         Attachments:, spatial.tar.gz,
> Local Lucene (Geo-search) has been donated to the Lucene project, per
 This issue is to handle the Lucene portion of integration.
> See

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message