lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Shai Erera (JIRA)" <>
Subject [jira] Commented: (LUCENE-2310) Reduce Fieldable, AbstractField and Field complexity
Date Tue, 16 Mar 2010 16:12:27 GMT


Shai Erera commented on LUCENE-2310:

That was usually the approach. You provide new methods, deprecate old ones, however both work
and not in a XOR mode. Both should work and we need to ensure that if people call both they
still function properly. Unless this has changed, in which case it should be clearly documented.

But I don't think it is a big problem to support both? If Document still keeps its fields
in a List then all should remain the same. We could have a 4.0 note to switch to a Map based
DS to better support remove, but that's questionable because we'll need to maintain ordering
on the fields (the order in which they inserted) though personally I don't think it should
matter much to the user, however that's the current implementation. 

> Reduce Fieldable, AbstractField and Field complexity
> ----------------------------------------------------
>                 Key: LUCENE-2310
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Sub-task
>          Components: Index
>            Reporter: Chris Male
>         Attachments: LUCENE-2310-Deprecate-AbstractField.patch, LUCENE-2310-Deprecate-AbstractField.patch,
> In order to move field type like functionality into its own class, we really need to
try to tackle the hierarchy of Fieldable, AbstractField and Field.  Currently AbstractField
depends on Field, and does not provide much more functionality that storing fields, most of
which are being moved over to FieldType.  Therefore it seems ideal to try to deprecate AbstractField
(and possible Fieldable), moving much of the functionality into Field and FieldType.

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