Skip to content

Fix flaky org.opensearch.knn.index.engine.InternalKNNEngineTests.testJVectorSearchStatsIncrement (followup) - #364

Merged
akash-shankaran merged 1 commit into
opensearch-project:mainfrom
reta:issues-283.2
Apr 7, 2026
Merged

Fix flaky org.opensearch.knn.index.engine.InternalKNNEngineTests.testJVectorSearchStatsIncrement (followup)#364
akash-shankaran merged 1 commit into
opensearch-project:mainfrom
reta:issues-283.2

Conversation

@reta

@reta reta commented Apr 6, 2026

Copy link
Copy Markdown
Collaborator

Description

Fix flaky org.opensearch.knn.index.engine.InternalKNNEngineTests.testJVectorSearchStatsIncrement (followup, disable refresh).

Related Issues

Followup on #359

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

….JVectorSearchStatsIncrement (followup)

Signed-off-by: Andriy Redko <drreta@gmail.com>
createKnnIndexMappingWithJVectorEngine(dimension, SpaceType.L2, VectorDataType.FLOAT);

// Disable refresh for the index
client().admin()

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@akash-shankaran this is one of the fixes I have tried but didn't include into original pull request, however it looks to be required.

@akash-shankaran
akash-shankaran merged commit 3b48096 into opensearch-project:main Apr 7, 2026
18 checks passed
opensearch-trigger-bot Bot pushed a commit that referenced this pull request Apr 7, 2026
….JVectorSearchStatsIncrement (followup) (#364)

Signed-off-by: Andriy Redko <drreta@gmail.com>
(cherry picked from commit 3b48096)
opensearch-trigger-bot Bot pushed a commit that referenced this pull request Apr 7, 2026
….JVectorSearchStatsIncrement (followup) (#364)

Signed-off-by: Andriy Redko <drreta@gmail.com>
(cherry picked from commit 3b48096)
reta added a commit that referenced this pull request Apr 7, 2026
….JVectorSearchStatsIncrement (followup) (#364) (#366)

(cherry picked from commit 3b48096)

Signed-off-by: Andriy Redko <drreta@gmail.com>
Co-authored-by: Andriy Redko <drreta@gmail.com>
reta added a commit that referenced this pull request Apr 7, 2026
….JVectorSearchStatsIncrement (followup) (#364) (#367)

(cherry picked from commit 3b48096)

Signed-off-by: Andriy Redko <drreta@gmail.com>
Co-authored-by: Andriy Redko <drreta@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants