Skip to content
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

FlintStreamingJobCleanerTask missing event listener #2574

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

vmmusings
Copy link
Member

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

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.

@vmmusings vmmusings marked this pull request as ready for review March 20, 2024 03:41
@vmmusings vmmusings changed the title FlintStreamingJobCleanerTask Implementation FlintStreamingJobCleanerTask missing event listener Mar 20, 2024
Copy link

codecov bot commented Mar 20, 2024

Codecov Report

Attention: Patch coverage is 0% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 95.36%. Comparing base (4dc83b7) to head (b6a22d3).

Files Patch % Lines
...sql/spark/cluster/ClusterManagerEventListener.java 0.00% 7 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2574      +/-   ##
============================================
- Coverage     95.40%   95.36%   -0.04%     
  Complexity     5129     5129              
============================================
  Files           490      490              
  Lines         14413    14419       +6     
  Branches        965      966       +1     
============================================
  Hits          13751    13751              
- Misses          637      643       +6     
  Partials         25       25              
Flag Coverage Δ
sql-engine 95.36% <0.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@vmmusings vmmusings self-assigned this Mar 20, 2024
@vmmusings vmmusings added bug Something isn't working v2.13.0 Issues targeting release v2.13.0 backport 2.x backport 2.13 labels Mar 20, 2024
@vmmusings vmmusings merged commit 284a0be into opensearch-project:main Mar 20, 2024
29 of 32 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Mar 20, 2024
Signed-off-by: Vamsi Manohar <[email protected]>
(cherry picked from commit 284a0be)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Mar 20, 2024
Signed-off-by: Vamsi Manohar <[email protected]>
(cherry picked from commit 284a0be)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
vmmusings pushed a commit that referenced this pull request Mar 20, 2024
(cherry picked from commit 284a0be)

Signed-off-by: Vamsi Manohar <[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>
vmmusings pushed a commit that referenced this pull request Mar 20, 2024
(cherry picked from commit 284a0be)

Signed-off-by: Vamsi Manohar <[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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x backport 2.13 bug Something isn't working v2.13.0 Issues targeting release v2.13.0
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants