Feat/#137 MultiQueue를 이용하여 Controller에서 오는 로그 요청의 부하 감소 #138
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- consume() 메서드에 대해 consumeLock 객체를 사용한 동기화 구현 - produce() 메서드의 동기화 제거로 불필요한 락킹 방지 - consume() 메서드의 대기 로직을 do-while 루프로 변경하여 spurious wakeup 방지 - produce() 메서드에서 consumeLock.notify() 호출로 정확한 스레드 깨우기 - 전반적인 동시성 처리 개선으로 생산자와 소비자 간 효율성 향상
- LockSupports를 활용한 Thread lock방식을 적용했습니다. - `volatile Thread consumerThread`에 대한 VarHandle을 적용했습니다.
- Producer, Consumer 방식을 적용해 리팩토링을 진행했습니다. - 이에 따라 사용하지 않는 테스트를 Deprecated 처리했습니다.
- LogQueue에 대한 Java-Doc을 추가했습니다.
- 사용하지 않는 메서드에 대한 Deprecated 처리를 진행했습니다. - AsyncLogProcessor에 대한 JavaDoc를 작성했습니다.
- `Consumer` -> `EventConsumer`, `Producer` -> `EventProducer` 이름 변경을 진행했습니다. - 패키지 위치를 변경했습니다.
- Deprecated 된 테스트를 제거했습니다.
- volatile 키워드 로 충분히 가시성이 보장
- LogQueue 비활성화
- 임시로 여러 큐를 가지는 MultiProcessor를 추가함
miiiinju1
requested review from
LuizyHub and
tidavid1
and removed request for
LuizyHub
|
Risk Level 3 - /home/runner/work/Team5-Guys/Team5-Guys/logbat/src/main/java/info/logbat/domain/log/queue/ReentrantLogQueue.java
Risk Level 3 - /home/runner/work/Team5-Guys/Team5-Guys/logbat/src/main/java/info/logbat/domain/log/queue/LogQueue.java
Risk Level 3 - /home/runner/work/Team5-Guys/Team5-Guys/logbat/src/main/java/info/logbat/domain/log/repository/MultiProcessor.java
🔒 Powered by Code Review GPT |
|
Git Conflict 제거 및 플로우 일원화를 위한 삭제 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚀 작업 내용
📸 이슈 번호
👀 Focus Commits [Optional]
✍ 궁금한 점