Skip to content

Muting broken test DatafusionSearchExecEngineTests - #21196

Merged
mch2 merged 2 commits into
opensearch-project:mainfrom
mch2:mutetest
Apr 10, 2026
Merged

Muting broken test DatafusionSearchExecEngineTests#21196
mch2 merged 2 commits into
opensearch-project:mainfrom
mch2:mutetest

Conversation

@mch2

@mch2 mch2 commented Apr 9, 2026

Copy link
Copy Markdown
Member

Description

mutes the broken test

Related Issues

related - #21195

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.

Signed-off-by: Marc Handalian <marc.handalian@gmail.com>
@mch2
mch2 requested a review from a team as a code owner April 9, 2026 22:14
@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

(Review updated until commit 047f441)

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ Recommended focus areas for review

Muted Test

The entire test class is muted with @AwaitsFix. This is a temporary workaround, but there should be a clear plan and timeline to fix the underlying issue and re-enable the tests. Leaving tests muted indefinitely can mask regressions.

@AwaitsFix(bugUrl = "https://github.com/opensearch-project/OpenSearch/issues/21195")
public class DatafusionSearchExecEngineTests extends OpenSearchTestCase {

@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

❌ Gradle check result for 0b89567: 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?

Signed-off-by: Marc Handalian <marc.handalian@gmail.com>
@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Persistent review updated to latest commit 047f441

@github-actions

Copy link
Copy Markdown
Contributor

✅ Gradle check result for 047f441: SUCCESS

@codecov

codecov Bot commented Apr 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.20%. Comparing base (056aac1) to head (047f441).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #21196      +/-   ##
============================================
+ Coverage     73.18%   73.20%   +0.01%     
+ Complexity    72934    72922      -12     
============================================
  Files          5885     5885              
  Lines        333174   333174              
  Branches      48065    48065              
============================================
+ Hits         243823   243885      +62     
+ Misses        69854    69771      -83     
- Partials      19497    19518      +21     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mch2
mch2 merged commit 60d267b into opensearch-project:main Apr 10, 2026
16 checks passed
aparajita31pandey pushed a commit to aparajita31pandey/OpenSearch that referenced this pull request Apr 18, 2026
…t#21196)

Signed-off-by: Aparajita Pandey <aparajita31pandey@gmail.com>
pradeep-L pushed a commit to pradeep-L/OpenSearch that referenced this pull request Apr 21, 2026
imRishN pushed a commit to imRishN/OpenSearch that referenced this pull request May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants