주문 큐 구현방식 변경, monitoring 설정 변경 #175
Merged
Conversation
This file contains hidden or 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
기존 PriorityBlockingQueue 기반의 주문 큐를, ConcurrentSkipListMap, ConcurrentSkipListSet을 사용한 구조로 변경했습니다.
매수 주문이 생성시간과 반대로 정렬되고 있었던 문제를 수정했습니다
temp
span으로부터 나온 trace정보로부터 metric을 추출해내는 설정을 반영했습니다
기존 jaeger가 인메모리 기반으로 동작하여 자주 종료되는 이슈가 있었기에 파일 기반으로 동작하도록 변경했습니다
추가 지표를 제공하도록 actuator yml 설정 jaeger가 터지지 않도록 sampling 비율 수정 jaeger가 셀프 모니터링시 오류가 발생하지 않도록 수정
hibernate가 너무 많은 지표를 전달하기 때문에, 얻고 싶은 지표만을 필터링합니다.
테스트 환경에서만 동작하고, local.properties의 secret에 기반해 동작하므로 안전합니다
wallet 생성 시점이 바뀐 후, 제대로 동작할 수 있도록 데이터 초기화 코드를 변경했습니다.
hibernate metric 지표가 warn 레벨로 찍히도록 수정, docker compose시 intellij 디버그용 에이전트를 포함하지 않도록 변경
기존 PriorityBlockingQueue에서 ConcurrentSkipListSet 기반으로 적용되도록 변경했습니다
|
caniro
approved these changes
Jun 23, 2025
Contributor
caniro
left a comment
caniro
left a comment
There was a problem hiding this comment.
불필요 설정 제거 및 otel collector 추가, 큐 구조 변경 고생하셨습니다.
주문 관련 작업과 체결 관련 작업 사이에 간섭이 훨씬 덜하겠네요.
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.
✨ 작업내용
🐞 이슈사항