-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update index random function to fix the bogus document deletion #11142
update index random function to fix the bogus document deletion #11142
Conversation
816d9f0
to
cfe13d8
Compare
Compatibility status:Checks if related components are compatible with change bab175a Incompatible componentsIncompatible components: [https://github.com/opensearch-project/performance-analyzer.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/reporting.git] |
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## main #11142 +/- ##
============================================
- Coverage 71.19% 71.15% -0.04%
+ Complexity 58816 58761 -55
============================================
Files 4883 4883
Lines 277144 277144
Branches 40281 40281
============================================
- Hits 197310 197202 -108
- Misses 63425 63465 +40
- Partials 16409 16477 +68 |
test/framework/src/main/java/org/opensearch/test/OpenSearchIntegTestCase.java
Show resolved
Hide resolved
test/framework/src/main/java/org/opensearch/test/OpenSearchIntegTestCase.java
Outdated
Show resolved
Hide resolved
cfe13d8
to
240ac29
Compare
Gradle Check (Jenkins) Run Completed with:
|
test/framework/src/main/java/org/opensearch/test/OpenSearchIntegTestCase.java
Outdated
Show resolved
Hide resolved
240ac29
to
06e157e
Compare
Gradle Check (Jenkins) Run Completed with:
|
#3603 - flaky test |
Signed-off-by: Neetika Singhal <[email protected]>
06e157e
to
bab175a
Compare
Gradle Check (Jenkins) Run Completed with:
|
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
) Signed-off-by: Neetika Singhal <[email protected]> (cherry picked from commit 2bd709b) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…nsearch-project#11142) Signed-off-by: Neetika Singhal <[email protected]> (cherry picked from commit 2bd709b) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: Neetika Singhal <[email protected]>
) (#11161) (cherry picked from commit 2bd709b) Signed-off-by: Neetika Singhal <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…nsearch-project#11142) Signed-off-by: Neetika Singhal <[email protected]>
…nsearch-project#11142) Signed-off-by: Neetika Singhal <[email protected]>
…nsearch-project#11142) Signed-off-by: Neetika Singhal <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
Description
This change updates the index random function to delete the bogus documents and add a refresh in the function such that the bogus documents are deleted and refresh ensures that the search doesnt account for the deleted segments.
This should fix some flaky issues encountered after adding the indexRadomForConcurrentSearch function to the tests.
Also, added TransportTwoNodesSearchIT.java to use indexRandomForConcurrentSearch.
Related Issues
Resolves #10900
Check List
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.