zacharymorn commented on code in PR #240:
URL: https://github.com/apache/lucene/pull/240#discussion_r1387438590
##########
lucene/core/src/java/org/apache/lucene/search/TopDocs.java:
##########
@@ -232,8 +232,8 @@ public static TopDocs merge(
/**
* Returns a new TopFieldDocs, containing topN results across the provided
TopFieldDocs, sorting
* by the specified {@link Sort}. Each of the TopDocs must have been sorted
by the same Sort, and
- * sort field values must have been filled (ie, <code>fillFields=true</code>
must be passed to
- * {@link TopFieldCollector#create}).
+ * sort field values must have been filled (ie, <code>fillFields=true</code>
must be passed to the
+ * constructor of {@link TopFieldCollectorManager}).
Review Comment:
Good catch! This most likely was an updated doc as
`TopFieldCollector#create` also did not take boolean field. Removed.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
