Skip to content

Conversation

@harshithasudhakar
Copy link
Member

Fixes #4263

Modifications

Restored interrupted states in lines 37 & 54

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (not applicable)
Copy link
Member

@Pil0tXia Pil0tXia left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@mxsm mxsm left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link

codecov bot commented Jul 30, 2023

Codecov Report

Merging #4297 (c5e361e) into master (4e8505b) will decrease coverage by 0.01%.
Report is 2 commits behind head on master.
The diff coverage is 88.46%.

❗ Current head c5e361e differs from pull request most recent head 504e590. Consider uploading reports for the commit 504e590 to get more accurate results

@@ Coverage Diff @@ ## master #4297 +/- ## ============================================ - Coverage 16.89% 16.88% -0.01%  Complexity 1428 1428 ============================================ Files 593 593 Lines 26056 26041 -15 Branches 2396 2380 -16 ============================================ - Hits 4402 4397 -5  - Misses 21214 21217 +3  + Partials 440 427 -13 
Files Changed Coverage Δ
...org/apache/eventmesh/common/utils/ThreadUtils.java 20.68% <0.00%> (-1.54%) ⬇️
...h/client/grpc/util/EventMeshCloudEventBuilder.java 78.90% <92.00%> (+7.18%) ⬆️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@xwm1992 xwm1992 merged commit dff6e7a into apache:master Aug 2, 2023
xuhongjia pushed a commit to Deckers-Ohana/eventmesh that referenced this pull request Mar 13, 2025
…eadUtils] (apache#4297) * Update ThreadUtils.java * Update ThreadUtils.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants