[jira] [Commented] (LUCENE-5870) Simplify StoredFieldsVisitor
[ https://issues.apache.org/jira/browse/LUCENE-5870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14085997#comment-14085997 ] Adrien Grand commented on LUCENE-5870: -- To give more context, a consequence of this change is that stored fields could store both ints and longs using a zlong without having to record whether it was an int or a long. Simplify StoredFieldsVisitor Key: LUCENE-5870 URL: https://issues.apache.org/jira/browse/LUCENE-5870 Project: Lucene - Core Issue Type: Improvement Reporter: Adrien Grand Assignee: Adrien Grand Fix For: 5.0, 4.10 Attachments: LUCENE-5870.patch StoredFieldVisitor has a visitor method for 4 numeric types: int, long, float and double. We should remove this specialization and just have a method that takes a java.lang.Number. -- This message was sent by Atlassian JIRA (v6.2#6252) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-5870) Simplify StoredFieldsVisitor
[ https://issues.apache.org/jira/browse/LUCENE-5870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14086313#comment-14086313 ] Ryan Ernst commented on LUCENE-5870: +1 Simplify StoredFieldsVisitor Key: LUCENE-5870 URL: https://issues.apache.org/jira/browse/LUCENE-5870 Project: Lucene - Core Issue Type: Improvement Reporter: Adrien Grand Assignee: Adrien Grand Fix For: 5.0, 4.10 Attachments: LUCENE-5870.patch StoredFieldVisitor has a visitor method for 4 numeric types: int, long, float and double. We should remove this specialization and just have a method that takes a java.lang.Number. -- This message was sent by Atlassian JIRA (v6.2#6252) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-5870) Simplify StoredFieldsVisitor
[ https://issues.apache.org/jira/browse/LUCENE-5870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14086433#comment-14086433 ] Uwe Schindler commented on LUCENE-5870: --- +1 In my opinion, I would use methods of Double/Float/... that directly return an instance, like Double.valueOf() instead of autoboxing Double.parseDouble() and so on. In 4.x we may still need some special case, because we have a backwards layer for early 3.x indexes there (like 3.2 or so). Simplify StoredFieldsVisitor Key: LUCENE-5870 URL: https://issues.apache.org/jira/browse/LUCENE-5870 Project: Lucene - Core Issue Type: Improvement Reporter: Adrien Grand Assignee: Adrien Grand Fix For: 5.0, 4.10 Attachments: LUCENE-5870.patch StoredFieldVisitor has a visitor method for 4 numeric types: int, long, float and double. We should remove this specialization and just have a method that takes a java.lang.Number. -- This message was sent by Atlassian JIRA (v6.2#6252) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-5870) Simplify StoredFieldsVisitor
[ https://issues.apache.org/jira/browse/LUCENE-5870?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14086463#comment-14086463 ] Robert Muir commented on LUCENE-5870: - I am a little concerned about this, since it results in loss of information. its similar to the parallel of removing TOKENIZED bit from stored fields before but yet still keeping StringField. This caused a lot of confusion for users. Today, the Analyzer doesn't know have full picture because of StringField/IntField/FloatField and company bypassing it. This causes a lot of pain, for example, you cannot even do a simple numeric range query with lucene without subclassing things with your own additional schema. In my opinion this stuff makes lucene too hard to use, because its too hard to reconstruct the doc from stored fields to e.g. perform an update to it and pass it back to indexwriter. Instead it tries to force people to either write/maintain a separate schema and subclass many things or force them to use some server that does this, which should not be necessary. An alternative would be, if we removed StringField/IntField/LongField etc and these were instead just KeywordAnalyzer/IntAnalyzer whatever in the analysis chain, then queryparser could form range queries without subclassing, queries on string fields would just work, and the schema needed to search would be implicit all in one place (the users Analyzer), making lucene a lot easier to use. Simplify StoredFieldsVisitor Key: LUCENE-5870 URL: https://issues.apache.org/jira/browse/LUCENE-5870 Project: Lucene - Core Issue Type: Improvement Reporter: Adrien Grand Assignee: Adrien Grand Fix For: 5.0, 4.10 Attachments: LUCENE-5870.patch StoredFieldVisitor has a visitor method for 4 numeric types: int, long, float and double. We should remove this specialization and just have a method that takes a java.lang.Number. -- This message was sent by Atlassian JIRA (v6.2#6252) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org