[SOLR-10399] generalize some SimpleFacet logic realted to PointsField vs other fields

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 6.6, 7.0
• Assignee: Chris M. Hostetter
• Reporter: Chris M. Hostetter
• Created: 31/Mar/17 22:58
• Updated: 08/Jun/17 23:26
• Resolved: 01/Apr/17 01:26

Description

in the below change, it looks like the logic that you apply to point fields would work in the general case and be as efficient?

+    if (ft.isPointField()) {
+      for (String term : terms) {
+        int count = searcher.numDocs(ft.getFieldQuery(null, sf, term), parsed.docs);
+        res.add(term, count);
+      }
+    } else {
+      for (String term : terms) {
+        String internal = ft.toInternal(term);
+        int count = searcher.numDocs(new TermQuery(new Term(field, internal)), parsed.docs);
+        res.add(term, count);
+      }
     }