You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Robert Muir (JIRA)" <ji...@apache.org> on 2016/03/07 17:55:40 UTC

[jira] [Commented] (LUCENE-7073) FieldType problems with Points

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

Robert Muir commented on LUCENE-7073:
-------------------------------------

There is a test for this copy constructor stuff, but it relies on hashcode/equals working, which is also broken.

It also has bugs in its reflection.

> FieldType problems with Points
> ------------------------------
>
>                 Key: LUCENE-7073
>                 URL: https://issues.apache.org/jira/browse/LUCENE-7073
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>
> The "copy constructor" is buggy and ignores the incoming points properties, because of a bug:
> {code}
>   public FieldType(FieldType ref) {
>     ...
>     // these two assignments have no effect. 
>     this.dimensionCount = dimensionCount;
>     this.dimensionNumBytes = dimensionNumBytes;
> {code}
> Another issue is that tokenized() returns true for all these point fields, which is bogus, and it confuses things like transitioning to points in some benchmark code (trying to clean up places still using LegacyNumeric* !!!). This code checks if (tokenized()) { try to use tokenstream }



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org