-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Fix flaky test in 130_script_fields_profile.yml #19356
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
Conversation
Signed-off-by: Binlong Gao <[email protected]>
❌ Gradle check result for d08649a: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
❌ Gradle check result for d08649a: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
❌ Gradle check result for d08649a: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
❌ Gradle check result for d08649a: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
❌ Gradle check result for 8a0a4ad: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #19356 +/- ##
============================================
- Coverage 72.84% 72.81% -0.03%
+ Complexity 69857 69806 -51
============================================
Files 5674 5674
Lines 320901 320901
Branches 46394 46394
============================================
- Hits 233756 233674 -82
- Misses 68205 68307 +102
+ Partials 18940 18920 -20 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…9356) Signed-off-by: Binlong Gao <[email protected]>
Description
From the failure message:
java.lang.AssertionError: Failure at [painless/130_script_fields_profile:69]: profile.shards.0.fetch.0.children.0.breakdown.set_next_reader_count didn't match expected value: profile.shards.0.fetch.0.children.0.breakdown.set_next_reader_count: expected Integer [1] but was Integer [2]
, there maybe two segments generated after indexing, so we need to forcemerge the index before search.Related Issues
Resolves #19243
Check List
- [ ] Functionality includes testing.
- [ ] API changes companion pull request created, if applicable.
- [ ] Public documentation issue/PR created, if applicable.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.