[jira] [Commented] (LUCENE-8041) All Fields.terms(fld) impls should be O(1) not O(log(N))

2019-05-16 Thread Huy Le (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16841902#comment-16841902 ] Huy Le commented on LUCENE-8041: I don't think that we can avoid paying the additional O(N) memory cost

[jira] [Commented] (LUCENE-8041) All Fields.terms(fld) impls should be O(1) not O(log(N))

2019-05-16 Thread Uwe Schindler (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16841123#comment-16841123 ] Uwe Schindler commented on LUCENE-8041: --- I have a maybe better suggestion instead of: - Build a

[jira] [Commented] (LUCENE-8041) All Fields.terms(fld) impls should be O(1) not O(log(N))

2019-05-16 Thread Huy Le (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16841075#comment-16841075 ] Huy Le commented on LUCENE-8041: I created a patch that use LinkedHashMap to maintain field name to

[jira] [Commented] (LUCENE-8041) All Fields.terms(fld) impls should be O(1) not O(log(N))

2018-06-12 Thread Robert Muir (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16510220#comment-16510220 ] Robert Muir commented on LUCENE-8041: - {quote} +1 to make term vectors consistent across the index;

[jira] [Commented] (LUCENE-8041) All Fields.terms(fld) impls should be O(1) not O(log(N))

2018-06-12 Thread David Smiley (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16510092#comment-16510092 ] David Smiley commented on LUCENE-8041: -- bq. I do wonder if an intermediate step would be to have a

[jira] [Commented] (LUCENE-8041) All Fields.terms(fld) impls should be O(1) not O(log(N))

2018-06-12 Thread Simon Willnauer (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16510047#comment-16510047 ] Simon Willnauer commented on LUCENE-8041: - bq. +1 to make term vectors consistent across the

[jira] [Commented] (LUCENE-8041) All Fields.terms(fld) impls should be O(1) not O(log(N))

2018-06-12 Thread Michael McCandless (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16510005#comment-16510005 ] Michael McCandless commented on LUCENE-8041: +1 to make term vectors consistent across the

[jira] [Commented] (LUCENE-8041) All Fields.terms(fld) impls should be O(1) not O(log(N))

2018-06-11 Thread Robert Muir (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16509023#comment-16509023 ] Robert Muir commented on LUCENE-8041: - {quote} That sounds like the cart leading the horse (allowing

[jira] [Commented] (LUCENE-8041) All Fields.terms(fld) impls should be O(1) not O(log(N))

2018-06-11 Thread David Smiley (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16508756#comment-16508756 ] David Smiley commented on LUCENE-8041: -- bq. I don't think we should remove field iteration/Fields

[jira] [Commented] (LUCENE-8041) All Fields.terms(fld) impls should be O(1) not O(log(N))

2018-06-11 Thread Robert Muir (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16508692#comment-16508692 ] Robert Muir commented on LUCENE-8041: - This has the downside that it sorts all fields on every call