jimczi commented on a change in pull request #1909:
URL: https://github.com/apache/lucene-solr/pull/1909#discussion_r492921270
##########
File path: lucene/core/src/java/org/apache/lucene/index/SortingCodecReader.java
##########
@@ -41,21 +41,13 @@
* {@link Sort}. This can be used to re-sort and index after it's been created
by wrapping all
* readers of the index with this reader and adding it to a fresh IndexWriter
via
* {@link IndexWriter#addIndexes(CodecReader...)}.
+ * NOTE: This reader should only be used for merging. Pulling fields from this
ready might be very costly and memory
Review comment:
nit:
```suggestion
* NOTE: This reader should only be used for merging. Pulling fields from
this reader might be very costly and memory
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]