jainankitk commented on issue #14536: URL: https://github.com/apache/lucene/issues/14536#issuecomment-2831945476
Currently, we use the `PointTreeBulkCollector` if we can collect the documents efficiently, irrespective of `DocValues` is indexed or not. IMO, we should be consistent in requiring doc values to be indexed for faceting to work. For example - it might be confusing for users to get exception if they try with bucket width of 100, and succeed when bucket width is 1000. In this specific case, it is fairly easy to support so I believe we should go ahead with that for consistency of functionality. Have created small PR - https://github.com/apache/lucene/pull/14559 for the same. Let me know what you think -- 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
