Skip to content

Commit 4275545

Browse files
committed
Fix indentation and update changelog
Signed-off-by: Medha Srinivasarao Srivatsa <medha.srivatsa@gmail.com>
1 parent b296817 commit 4275545

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
1010
* Enable derived sources for vectors to save storage costs. [241] (https://github.com/opensearch-project/opensearch-jvector/pull/241)
1111
### Enhancements
1212
### Bug Fixes
13+
* Fix flaky test KNNJVectorTests.testJVectorKnnIndex_simpleCase
1314
### Infrastructure
1415
* Upgrade Lucene to 10.4.0 [292] (https://github.com/opensearch-project/opensearch-jvector/pull/292)
1516
### Documentation

src/test/java/org/opensearch/knn/index/codec/jvector/KNNJVectorTests.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,10 @@ public void testJVectorKnnIndex_simpleCase() throws IOException {
7575
w.commit();
7676

7777
try (IndexReader reader = DirectoryReader.open(w)) {
78-
int expectedNumOfSegments = indexWriterConfig.getMaxBufferedDocs() < 0 ? 1 : totalNumberOfDocs > indexWriterConfig.getMaxBufferedDocs() ? totalNumberOfDocs/indexWriterConfig.getMaxBufferedDocs() + 1 : 1;
78+
int expectedNumOfSegments = indexWriterConfig.getMaxBufferedDocs() < 0 ? 1
79+
: totalNumberOfDocs > indexWriterConfig.getMaxBufferedDocs()
80+
? totalNumberOfDocs / indexWriterConfig.getMaxBufferedDocs() + 1
81+
: 1;
7982
log.info("We should now have a {} segment(s) with 10 documents", expectedNumOfSegments);
8083
Assert.assertEquals(expectedNumOfSegments, reader.getContext().leaves().size());
8184
Assert.assertEquals(totalNumberOfDocs, reader.numDocs());

0 commit comments

Comments
 (0)