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

[PinpiontWeb] fix a ActiveThread concurrency problem #1482

Closed
koo-taejin opened this issue Jan 25, 2016 · 0 comments
Closed

[PinpiontWeb] fix a ActiveThread concurrency problem #1482

koo-taejin opened this issue Jan 25, 2016 · 0 comments
Assignees
@koo-taejin
Labels
bug
Milestone
1.5.2

Comments

@koo-taejin
Copy link
Member

If multiple threads invokes WebSocketSession's sendMessage method at the same time, ;
then it is possible to makes IllegalStateException.
@koo-taejin koo-taejin added the bug label Jan 25, 2016
@koo-taejin koo-taejin self-assigned this Jan 25, 2016
@koo-taejin koo-taejin added this to the 1.5.2 milestone Jan 26, 2016
koo-taejin added a commit to koo-taejin/pinpoint that referenced this issue Jan 26, 2016
@koo-taejin
[PinpiontWeb] fix a ActiveThread concurrency problem pinpoint-apm#1482
0d3b9da 
1. changed to catched Excpetion Type in to throw WebSocketSession . (IOException -> Exception)
2. changed log level.
koo-taejin added a commit that referenced this issue Jan 26, 2016
@koo-taejin
Merge pull request #1483 from koo-taejin/#1482
6be607d 
[PinpiontWeb] fix a ActiveThread concurrency problem #1482
koo-taejin added a commit to koo-taejin/pinpoint that referenced this issue Jan 29, 2016
@koo-taejin
[PinpiontWeb] fix a ActiveThread concurrency problem pinpoint-apm#1482
967c69e 
1. added OrderedThreadPoolExecutor.
2. separated thread from WebSocketSessionId.
koo-taejin added a commit to koo-taejin/pinpoint that referenced this issue Jan 30, 2016
@koo-taejin
Merge pull request #3 from emeroad/pinpoint-apm#1482
4dbb817 
add SimpleOrderedThreadPool pinpoint-apm#1482
koo-taejin added a commit to koo-taejin/pinpoint that referenced this issue Jan 30, 2016
@koo-taejin
[PinpiontWeb] fix a ActiveThread concurrency problem pinpoint-apm#1482
e0f7a0e 
  1. applied SimpleThreadPoolExecutor for separated thread from WebSocketSessionId.
emeroad added a commit that referenced this issue Feb 1, 2016
@emeroad
Merge pull request #1502 from koo-taejin/#1482_apply_ordered_thread_pool
6ee14bd 
#1482 apply ordered thread pool
koo-taejin added a commit to koo-taejin/pinpoint that referenced this issue Oct 18, 2016
@koo-taejin
[PinpiontWeb] fix a ActiveThread concurrency problem pinpoint-apm#1482
88d5fe5 
If webSocketSession status is not opened, then do not binding to flush aggregator.
koo-taejin added a commit that referenced this issue Oct 18, 2016
@koo-taejin
[PinpiontWeb] fix a ActiveThread concurrency problem #1482
4cf078d 
If webSocketSession status is not opened, then do not binding to flush aggregator.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@koo-taejin koo-taejin

Labels
bug
Projects
None yet
Milestone
1.5.2
Development

No branches or pull requests
1 participant
@koo-taejin