Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement] InterruptedExceptions should never be ignored[ThreadUtils].

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.