Skip to content

Bugfix: fixed the following bugs #227

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 16, 2025
Merged

Bugfix: fixed the following bugs #227

merged 1 commit into from
Sep 16, 2025

Conversation

@chhy2009
Copy link
Contributor

@chhy2009 chhy2009 commented Sep 9, 2025 •
edited
Loading

  • fix the issue that program will cause a core dump at Promise::SetValue rarely when make rpc call from handle or io work thread
  • fix MQThreadPool memory leak when queue is full
  • fix the issue where the timeout parameter of HTTP asynchronous streaming ReadFullResponse interface did not take effect
  • fix the issue that stream length is 0 in HttpServiceProxy streaming request
  • correct type of timeout from int to uint32_t in http stream to avoid overflow during conversion
  • make tvar sampler remain valid before and after SamplerCollectorStop/Start

@chhy2009 chhy2009 force-pushed the fix_bug branch 2 times, most recently from 5703773 to 98009e8 Compare September 10, 2025 03:17
@chhy2009 chhy2009 closed this Sep 10, 2025
@chhy2009 chhy2009 reopened this Sep 10, 2025
@chhy2009 chhy2009 closed this Sep 10, 2025
@chhy2009 chhy2009 reopened this Sep 10, 2025
@chhy2009 chhy2009 closed this Sep 10, 2025
@chhy2009 chhy2009 reopened this Sep 10, 2025
@chhy2009 chhy2009 closed this Sep 10, 2025
@chhy2009 chhy2009 reopened this Sep 10, 2025
@chhy2009 chhy2009 force-pushed the fix_bug branch 2 times, most recently from 98009e8 to 77324a4 Compare September 11, 2025 11:16
@chhy2009
Bugfix: fixed the following bugs
402bfae 
- fix the issue that program will cause a core dump at Promise::SetValue rarely when make rpc call from handle or io work thread
- fix MQThreadPool memory leak when queue is full
- fix the issue where the timeout parameter of HTTP asynchronous streaming ReadFullResponse interface did not take effect
- fix the issue that stream length is 0 in HttpServiceProxy streaming request
- correct type of timeout from int to uint32_t in http stream to avoid overflow during conversion
- make tvar sampler remain valid before and after SamplerCollectorStop/Start
@helloopenworld helloopenworld merged commit 0e5a67f into trpc-group:main Sep 16, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@helloopenworld helloopenworld helloopenworld approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@chhy2009 @helloopenworld