Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r826455347
########## File path: lucene/facet/src/java/org/apache/lucene/facet/Facets.java ########## @@ -48,4 +48,13 @@ public abstract FacetResult getTopChildren(int topN, String dim, String... path) * indexed, for example depending on the type of document. */ public abstract List<FacetResult> getAllDims(int topN) throws IOException; + + /** + * Returns labels for topN dimensions and their topNChildren sorted by the number of hits that + * dimension matched + */ + public List<FacetResult> getTopDims(int topNDims, int topNChildren) throws IOException { Review comment: My current javadoc does not well describe this new functionality, and I will add more to it. Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org