[
https://issues.apache.org/jira/browse/LUCENE-1219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12622502#action_12622502
]
Michael McCandless commented on LUCENE-1219:
--------------------------------------------
OK this patch looks good! I plan to commit in a day or two.
{quote}
This is one nice feature, but I permanently had a feeling I do not understand
this Field structures, roles and responsibilities
Field/Fieldable/AbstractField hierarchy is really ripe for good
re-factoring.This bigamy with index / search use cases makes things not really
easy to follow, Hoss has right, we need some way to divorce RetrievedField from
FieldToBeIndexed, they are definitely not the same, just very similar.
{quote}
I completely agree!
> support array/offset/ length setters for Field with binary data
> ---------------------------------------------------------------
>
> Key: LUCENE-1219
> URL: https://issues.apache.org/jira/browse/LUCENE-1219
> Project: Lucene - Java
> Issue Type: Improvement
> Components: Index
> Reporter: Eks Dev
> Assignee: Michael McCandless
> Priority: Minor
> Attachments: LUCENE-1219.extended.patch, LUCENE-1219.extended.patch,
> LUCENE-1219.patch, LUCENE-1219.patch, LUCENE-1219.patch, LUCENE-1219.patch,
> LUCENE-1219.take2.patch, LUCENE-1219.take3.patch
>
>
> currently Field/Fieldable interface supports only compact, zero based byte
> arrays. This forces end users to create and copy content of new objects
> before passing them to Lucene as such fields are often of variable size.
> Depending on use case, this can bring far from negligible performance
> improvement.
> this approach extends Fieldable interface with 3 new methods
> getOffset(); gettLenght(); and getBinaryValue() (this only returns reference
> to the array)
>
--
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
