[
https://issues.apache.org/jira/browse/SOLR-4085?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13508629#comment-13508629
]
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 java.io.Reader!)
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: https://issues.apache.org/jira/browse/SOLR-4085
> 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: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
