Skip to content

Conversation

@xjlgod
Copy link
Contributor

@xjlgod xjlgod commented May 13, 2025

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

feat: add io-uring to server and client

Ⅱ. Does this pull request fix one issue?

fix #7063

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@codecov
Copy link

codecov bot commented May 13, 2025

Codecov Report

❌ Patch coverage is 20.00000% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.39%. Comparing base (dc9bf00) to head (9f7ebdb).

Files with missing lines Patch % Lines
...che/seata/core/rpc/netty/NettyServerBootstrap.java 9.09% 10 Missing ⚠️
...g/apache/seata/core/rpc/netty/NettyBaseConfig.java 14.28% 2 Missing and 4 partials ⚠️
...che/seata/core/rpc/netty/NettyClientBootstrap.java 33.33% 4 Missing ⚠️
...utoconfigure/properties/TransportIoProperties.java 25.00% 3 Missing ⚠️
...apache/seata/core/rpc/netty/NettyServerConfig.java 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                2.x    #7338      +/-   ##
============================================
- Coverage     62.41%   62.39%   -0.02%     
  Complexity      721      721              
============================================
  Files          1324     1325       +1     
  Lines         50125    50153      +28     
  Branches       5920     5924       +4     
============================================
+ Hits          31285    31295      +10     
- Misses        16027    16046      +19     
+ Partials       2813     2812       -1     
Files with missing lines Coverage Δ
...ava/org/apache/seata/common/ConfigurationKeys.java 0.00% <ø> (ø)
...in/java/org/apache/seata/common/DefaultValues.java 100.00% <ø> (ø)
...toconfigure/SeataCoreEnvironmentPostProcessor.java 100.00% <100.00%> (ø)
...ta/spring/boot/autoconfigure/StarterConstants.java 100.00% <ø> (ø)
...apache/seata/core/rpc/netty/NettyServerConfig.java 72.28% <0.00%> (-0.89%) ⬇️
...utoconfigure/properties/TransportIoProperties.java 25.00% <25.00%> (ø)
...che/seata/core/rpc/netty/NettyClientBootstrap.java 58.41% <33.33%> (-1.59%) ⬇️
...g/apache/seata/core/rpc/netty/NettyBaseConfig.java 71.05% <14.28%> (-7.74%) ⬇️
...che/seata/core/rpc/netty/NettyServerBootstrap.java 67.59% <9.09%> (-6.64%) ⬇️

... and 10 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

xjlgod added 13 commits May 13, 2025 22:05
# Conflicts:
#	changes/en-us/2.x.md
#	changes/zh-cn/2.x.md
#	common/src/main/java/org/apache/seata/common/ConfigurationKeys.java
#	core/src/main/java/org/apache/seata/core/rpc/netty/NettyBaseConfig.java
#	core/src/main/java/org/apache/seata/core/rpc/netty/NettyClientBootstrap.java
#	core/src/main/java/org/apache/seata/core/rpc/netty/NettyServerBootstrap.java
#	core/src/test/java/org/apache/seata/core/rpc/netty/NettyClientBootstrapTest.java
#	script/config-center/config.txt
#	seata-spring-autoconfigure/seata-spring-autoconfigure-core/src/main/java/org/apache/seata/spring/boot/autoconfigure/SeataCoreEnvironmentPostProcessor.java
#	seata-spring-autoconfigure/seata-spring-autoconfigure-core/src/main/java/org/apache/seata/spring/boot/autoconfigure/StarterConstants.java
# Conflicts:
#	changes/en-us/2.x.md
#	changes/zh-cn/2.x.md
#	common/src/main/java/org/apache/seata/common/DefaultValues.java
# Conflicts:
#	changes/en-us/2.x.md
#	core/src/main/java/org/apache/seata/core/rpc/netty/NettyBaseConfig.java
#	core/src/test/java/org/apache/seata/core/rpc/netty/NettyClientBootstrapTest.java
# Conflicts:
#	core/src/main/java/org/apache/seata/core/rpc/netty/NettyBaseConfig.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.

support IO_uring feature

1 participant