thecoop commented on code in PR #14510:
URL: https://github.com/apache/lucene/pull/14510#discussion_r2071769318
##########
lucene/backward-codecs/src/java/org/apache/lucene/backward_codecs/lucene50/Lucene50CompoundReader.java:
##########
@@ -74,7 +74,7 @@ public Lucene50CompoundReader(Directory directory,
SegmentInfo si) throws IOExce
}
expectedLength += CodecUtil.footerLength();
- handle = directory.openInput(dataFileName,
IOContext.DEFAULT.withReadAdvice(ReadAdvice.NORMAL));
+ handle = directory.openInput(dataFileName,
IOContext.DEFAULT.withHints(FileTypeHint.DATA));
Review Comment:
Here, the handle isn't really used direct - its used to slice a new
IndexInput, but then a replacement `IOContext` is specified there. So we don't
actually need a hint/readadvice at all here?
--
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
