lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adrien Grand (JIRA)" <>
Subject [jira] [Commented] (SOLR-4085) Commit-free ExternalFileField
Date Mon, 03 Dec 2012 10:27:58 GMT


Adrien Grand commented on SOLR-4085:

To me ExternalFileField is a convenient (but hacky) way to map documents to floats values
in a static index. I understand that it may need to support updates (ReloadCacheRequestHandler)
in order to seldom fix values.

Commit-free updates support however makes ExternalFileField look like the perfect candidate
to support frequent document updates but I don't like the fact that it needs to read the whole
file every time (with a!) and resolve IDs against the top-level reader. Moreover
the lack of atomicity makes it different from everything else we have in Lucene and Solr and
I imagine people having a scoring function such as if (fieldA > 0) { return fieldB / fieldA
} else { return 0 }: this might break badly if fieldA is an ExternalFileField and its valued
is modified from 1 to 0 between the first and the second evaluation. I'm also a little worried
about the problems you mentioned regarding concurrent reloads.

But on the other hand you didn't modify anything outside of ExternalFileField and FileFloatSource
so it's perfectly fine with me if Alan or another committer decides this should be committed.

> Commit-free ExternalFileField
> -----------------------------
>                 Key: SOLR-4085
>                 URL:
>             Project: Solr
>          Issue Type: Improvement
>          Components: Schema and Analysis
>    Affects Versions: 4.1
>            Reporter: Mikhail Khludnev
>              Labels: externalfilefield
>         Attachments: SOLR-4085.patch
> Let's reload ExternalFileFields without commit!

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

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

View raw message