Skip to content

Conversation

@DMwangnima
Copy link
Contributor

@DMwangnima DMwangnima commented Jul 14, 2025

What type of PR is this?

feat

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.

feat(gRPC): 支持查看 HTTP2 Client 的最大并发流数量

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):
Client 在创建 Stream 时受 HTTP2 Settings MaxConcurrentStreams 制约,若当前连接上的并发流超过这个限制,则 NewStream 会被阻塞。
支持查看当前状态的 MaxConcurrentStreams 以更好地排查问题。

(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

@DMwangnima DMwangnima requested review from a team as code owners July 14, 2025 12:23
@codecov
Copy link

codecov bot commented Jul 14, 2025

Codecov Report

❌ Patch coverage is 92.59259% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.21%. Comparing base (ca63109) to head (6c25edd).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
pkg/remote/trans/nphttp2/grpc/http2_client.go 92.59% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1820      +/-   ##
==========================================
- Coverage   60.26%   60.21%   -0.06%     
==========================================
  Files         381      381              
  Lines       34447    34463      +16     
==========================================
- Hits        20759    20751       -8     
- Misses      12427    12449      +22     
- Partials     1261     1263       +2     
Flag Coverage Δ
integration 48.57% <0.00%> (-0.03%) ⬇️
unit 50.75% <92.59%> (-0.02%) ⬇️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

HeyJavaBean
HeyJavaBean previously approved these changes Jul 15, 2025
@DMwangnima DMwangnima force-pushed the feat/grpc_max_streams branch 2 times, most recently from 0649699 to 27a3cb5 Compare July 15, 2025 06:53
@DMwangnima DMwangnima force-pushed the feat/grpc_max_streams branch from 27a3cb5 to 6c25edd Compare August 29, 2025 06:43
@DMwangnima DMwangnima merged commit 9bc7b48 into cloudwego:main Aug 29, 2025
25 checks passed
@DMwangnima DMwangnima deleted the feat/grpc_max_streams branch August 29, 2025 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants