Skip to content
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.

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

fix ignore SIGPIPE signal concurrent issue with other library #2301

Merged
merged 1 commit into from
Jul 10, 2023

Conversation

lrita
Copy link
Contributor

@lrita lrita commented Jul 4, 2023

What problem does this PR solve?

Issue Number:

Problem Summary:
There is a crash issue that at beginning many library concurrently ignore SIGPIPE to assert failed.

What is changed and the side effects?

none
Changed:
none
Side effects:

  • Performance effects(性能影响):
    none
  • Breaking backward compatibility(向后兼容性):
    none

@wwbmmm
Copy link
Contributor

wwbmmm commented Jul 10, 2023

LGTM

@serverglen serverglen merged commit c6d51ee into apache:master Jul 10, 2023
yanglimingcn pushed a commit to yanglimingcn/brpc that referenced this pull request Oct 31, 2023
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.

3 participants