Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@zsp419
Copy link
Contributor

@zsp419 zsp419 commented Feb 21, 2023

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

使AbstractHttpExecutor类支持PUT方式的请求

Ⅱ. Does this pull request fix one issue?

fixes #5067

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@codecov-commenter
Copy link

codecov-commenter commented Feb 21, 2023

Codecov Report

Merging #5377 (ea8aeb9) into 2.x (90fac5e) will decrease coverage by 0.03%.
The diff coverage is 50.00%.

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

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #5377      +/-   ##
============================================
- Coverage     48.50%   48.48%   -0.03%     
+ Complexity     4303     4302       -1     
============================================
  Files           775      775              
  Lines         27397    27407      +10     
  Branches       3413     3414       +1     
============================================
- Hits          13290    13289       -1     
- Misses        12701    12713      +12     
+ Partials       1406     1405       -1     
Impacted Files Coverage Δ
...o/seata/integration/http/AbstractHttpExecutor.java 55.55% <50.00%> (-8.60%) ⬇️
...erver/storage/file/session/FileSessionManager.java 61.14% <0.00%> (-0.64%) ⬇️
...rage/redis/store/RedisTransactionStoreManager.java 77.12% <0.00%> (-0.26%) ⬇️
...er/src/main/java/io/seata/server/ServerRunner.java 50.00% <0.00%> (ø)

@zsp419 zsp419 mentioned this pull request Feb 21, 2023
1 task
@slievrly slievrly added this to the 2.0.0 milestone Feb 21, 2023
Copy link
Member

@slievrly slievrly 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

@slievrly slievrly left a comment

Choose a reason for hiding this comment

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

@zsp419
Copy link
Contributor Author

zsp419 commented Feb 21, 2023

@slievrly
Copy link
Member

@slievrly slievrly merged commit 1c778e3 into apache:2.x Feb 21, 2023
YvCeung pushed a commit to YvCeung/incubator-seata that referenced this pull request Dec 25, 2025
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.

http工具类AbstractHttpExecutor中,缺少PUT方式的方法

3 participants