[ 
https://issues.apache.org/jira/browse/LUCENE-4101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13288157#comment-13288157
 ] 

Robert Muir commented on LUCENE-4101:
-------------------------------------

{quote}
I set it to blocker because I think the 4.0 API for creating stored
fields is confusing now (separate from the changes being explored in
LUCENE-3312).
This (XXXField/FieldType) is one of the major changed APIs in 4.0 so
it's worth spending effort reviewing/fixing it before we ship it. The
same goes for all our other new AIPs in 4.0: we should scrutinize them.
{quote}

OK: I totally agree with scrutinizing the API. I also feel bad about
making the comment here because I know its really a ton of work to make
patches like this... and delays can cause lots of merging grief.

{quote}
personally i dont like the fact you have to add the field twice.
Expert users don't have to (they can make a custom field type, at least
until 5.0). This API is just for basic usage, and I think it's
important that API be really easy to use.
{quote}

I think this is where my concern lies? I think its the wrong way around:
basic users should be able to add the field once and just say its indexed & 
stored :)

I agree that TYPE_STORED is confusing, and I also agree that StoredXXX is not 
great.
I guess i just want to know that we've had a little time to think about the 
different
options here, maybe there is a better solution.
                
> Remove XXXField.TYPE_STORED
> ---------------------------
>
>                 Key: LUCENE-4101
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4101
>             Project: Lucene - Java
>          Issue Type: Bug
>            Reporter: Michael McCandless
>            Assignee: Michael McCandless
>            Priority: Blocker
>             Fix For: 4.0, 5.0
>
>         Attachments: LUCENE-4101.patch
>
>
> Spinoff from LUCENE-3312.
> For 4.0 I think we should simplify the sugar field APIs by requiring
> that you add a StoredField if you want to store the field.  Expert users
> can still make a custom FieldType that both stores and indexes...

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
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]

Reply via email to