Skip to content

Commit 6a8143a

Browse files
committed
update after review
1 parent 5d74ec2 commit 6a8143a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene60/MetadataOnlyBKDReader.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,12 +87,12 @@ public MetadataOnlyBKDReader(IndexInput metaIn, boolean isVersionPost86) throws
8787
docCount = metaIn.readVInt();
8888

8989
// The pre-8.6 code does not read the following fields that its standard Lucene counterpart does. After experimenting with the
90-
// code, we got to the assumption that these are the last fields being read, which are not needed in the metadata-only reader, and
90+
// code, we got to the conclusion that these are the last fields being read, which are not needed in the metadata-only reader, and
9191
// we can safely ignore them when loading the file. Although by coincidence, nothing breaks if we read a couple of VLongs, as long
9292
// as some bytes are available to read.
9393
//
9494
// The extra reads have been introduced to process IndexInput created with Lucene86Codec+, where a new BKD format has been
95-
// introduced. It seems that we have stricter checks around the header and footer starting from the 86 formats hence we do need to
95+
// introduced. We have stricter checks around the header and footer starting from the 86 formats hence we do need to
9696
// consume all the data input there but not in previous formats.
9797
//
9898
// For correctness, we added version checking here. If and only if, the version is 8.6 or higher, we read the additional fields.

0 commit comments

Comments
 (0)