Skip to content

[Branch-2.9][fix][client] Fix MaxQueueSize semaphore release leak in createOpSendMsg. - #16958

Merged
Technoboy- merged 1 commit into
apache:branch-2.9from
Nicklee007:chrry-pick-PR16915-PR16837-to-branch-2.9
Aug 9, 2022
Merged

[Branch-2.9][fix][client] Fix MaxQueueSize semaphore release leak in createOpSendMsg. #16958
Technoboy- merged 1 commit into
apache:branch-2.9from
Nicklee007:chrry-pick-PR16915-PR16837-to-branch-2.9

Conversation

@Nicklee007

Copy link
Copy Markdown
Contributor

Motivation

This PR is part of work of cherry-pick #16915 to branch 2.9
Release MaxQueueSize semaphore when invalid message.

Modifications

  1. add semaphore release;
  2. fix unit test;

Documentation

  • doc-not-needed
    (Please explain why)

@Nicklee007 Nicklee007 changed the title cherry pick #16915 to branch 2.9 [Branch-2.9][fix][client] Fix MaxQueueSize semaphore release leak in createOpSendMsg. Aug 5, 2022
@codelipenghui

Copy link
Copy Markdown
Contributor

@Nicklee007 Please help resolve the conflicts

@Nicklee007
Nicklee007 force-pushed the chrry-pick-PR16915-PR16837-to-branch-2.9 branch from 1ec9e3d to 2fd6ac9 Compare August 8, 2022 12:18
@Nicklee007
Nicklee007 force-pushed the chrry-pick-PR16915-PR16837-to-branch-2.9 branch from 2fd6ac9 to 4c9a410 Compare August 8, 2022 12:21
@Nicklee007

Copy link
Copy Markdown
Contributor Author

@Nicklee007 Please help resolve the conflicts

@codelipenghui resolved, PTAL

@Technoboy-
Technoboy- merged commit f8ce46e into apache:branch-2.9 Aug 9, 2022
@mattisonchao

Copy link
Copy Markdown
Member

@Nicklee007 Very thanks to you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-picked/branch-2.9 Archived: 2.9 is end of life release/2.9.4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants