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

Adrien Grand commented on LUCENE-8870:
--------------------------------------

I think this new constructor would be misleading. For instance it might be 
tempting to use if you wanted to index doubles. But the only thing you can 
index with this constructor is the string representation of the double values, 
which is unlikely to be helpful.

I wonder whether we should make this class abstract instead so that it can't be 
instantiated directly, and potentially enhance some of its sub classes to 
address use-cases that were only doable with this Field class until now, such 
as having a text field with term vectors enabled.

> Support numeric value in Field class
> ------------------------------------
>
>                 Key: LUCENE-8870
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8870
>             Project: Lucene - Core
>          Issue Type: New Feature
>            Reporter: Namgyu Kim
>            Priority: Major
>         Attachments: LUCENE-8870.patch
>
>
> I checked the following comment in Field class.
> {code:java}
> // TODO: allow direct construction of int, long, float, double value too..?
> {code}
> We already have some fields like IntPoint and StoredField, but I think it's 
> okay.
> The test cases are set in the TestField class.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to