Feat/#134 LogQueue의 부하를 감소하도록 Producer Consumer 구조로 최적화

[miiiinju1]

🚀 작업 내용

• LogQueue의 동기화 메커니즘 최적화
  • consume() 메서드에 대해 consumeLock 객체를 사용한 동기화 구현
  • produce() 메서드의 동기화 제거로 불필요한 락킹 방지
  • consume() 메서드의 대기 로직을 do-while 루프로 변경하여 spurious wakeup 방지
  • produce() 메서드에서 consumeLock.notify() 호출로 정확한 스레드 깨우기
  • 전반적인 동시성 처리 개선으로 생산자와 소비자 간 효율성 향상

📸 이슈 번호

• close [💡 FEAT] LogQueue 최적화: Producer Consumer 구조 도입 #134

👀 Focus Commits [Optional]

• 커밋해시: 내용

✍ 궁금한 점

• 중점적으로 봐줄 내용
• 변수명 괜찮나요
• 로직이 좀 더럽나요
• 가독성이 좀 그런가요?

[github-actions]

Risk Level 2 - /home/runner/work/Team5-Guys/Team5-Guys/logbat/src/main/java/info/logbat/domain/log/application/LogService.java

Consider handling specific exceptions instead of a generic Exception in the catch block. This improves error handling and debugging. Example:

} catch (SpecificException e) {
    log.error(\"Specific error occurred: {}\", request, e);
}

---

Risk Level 3 - /home/runner/work/Team5-Guys/Team5-Guys/logbat/src/main/java/info/logbat/domain/log/queue/LogQueue.java

The implementation of the consume method could lead to potential performance issues due to busy waiting. Consider using a more efficient waiting mechanism. Example:

while (queue.isEmpty()) {
    LockSupport.parkNanos(timeoutNanos);
}

---

Risk Level 3 - /home/runner/work/Team5-Guys/Team5-Guys/logbat/src/main/java/info/logbat/domain/log/queue/ReentrantLogQueue.java

The consume method should handle InterruptedException more gracefully. Instead of throwing a RuntimeException, consider restoring the interrupt status. Example:

} catch (InterruptedException e) {
    Thread.currentThread().interrupt();
}

---

🔍⚠️🔧

---

Powered by Code Review GPT

[tidavid1]

정합성 이슈가 터졌네요... 해결해봅시다 :C

[LuizyHub]

정합성 같이 손봐요!