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.

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

Revise synchronized blocks/methods in critical code path to avoid virtual threads pinning #8643

Closed
artembilan opened this issue Jun 8, 2023 · 0 comments · Fixed by #8652
Closed

Revise synchronized blocks/methods in critical code path to avoid virtual threads pinning #8643

artembilan opened this issue Jun 8, 2023 · 0 comments · Fixed by #8652
Assignees
@artembilan @tzolov
Labels
type: enhancement
Milestone
6.2.0-M1

Comments

@artembilan
Copy link
Member

artembilan commented Jun 8, 2023 •
edited
Loading

No description provided.

@artembilan artembilan added this to the 6.2.x milestone Jun 8, 2023
tzolov added a commit to tzolov/spring-integration that referenced this issue Jun 16, 2023
@tzolov
First pass - trivial synchronized blocks
e100e5e 
  - Convert the "trivial" synchronized block into reetrant locks.

 Resolves spring-projects#8643
@tzolov tzolov mentioned this issue Jun 16, 2023
Merged
tzolov added a commit to tzolov/spring-integration that referenced this issue Jun 16, 2023
@tzolov
First pass - trivial synchronized blocks
d8a7eff 
  - Convert the "trivial" synchronized block into reetrant locks.

 Resolves spring-projects#8643
@artembilan artembilan modified the milestones: 6.2.x, 6.2.0-M1 Jun 21, 2023
artembilan pushed a commit that referenced this issue Jun 21, 2023
@tzolov
GH-8643: Replace synchronized with Lock
c38ed96 
Fixes #8643

* First pass - trivial synchronized blocks
  - Convert the "trivial" `synchronized` block into `ReentrantLock`.

* fix checkstyle

* use blocking lock

* Secon pass - handle multi-lock cases

* javadoc + year

* addres first batch of review suggestions

* fix checkstyle issues

* fix the mqtt parent/child lock monitor sharing

* fix the mqtt parent/child lock monitor sharing, v2

* patch the stomp test
@flex-myeonghyeon flex-myeonghyeon mentioned this issue Sep 18, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@artembilan artembilan

@tzolov tzolov

Labels
type: enhancement
Projects
None yet
Milestone
6.2.0-M1
Development

Successfully merging a pull request may close this issue.

First pass - trivial synchronized blocks tzolov/spring-integration
2 participants
@artembilan @tzolov