Skip to content

Conversation

@109an94
Copy link
Contributor

@109an94 109an94 commented Jun 10, 2025

✨ 작업내용

  • NPE문제 해결 70cf64d
  • actuator 및 prometheus 설정값 등록 220b8f9
  • 데이터 정합성 최적화 918b265
  • realitybot 시계열 데이터 저장으로 인한 오버헤드 제거 56e37f7

🐞 이슈사항

이슈 번호 제목 상태
#000 볼륨 0 입력 문제 9b9672e ✅ 해결

⚠️ 특별사항

  • 예: DB 마이그레이션 필요
  • meterregistry에 의한 메트릭 등록 최적화를 위해 noopmeterregistry 구현을 하다가 실패해서 제 realitybot 의 주문금액을 기록하는 걸 최적화했습니다.
  • simplemeterregistry가 noop을 대체한다는 이야기가 있긴 했었는데 bean에 등록되면 작동하는 건 매한가지라고 해서 최적화 방향으로 갔습니다.
  • **혹시 trade쪽이 최신인지 확인부탁드립니다..
  • 쓰레드 슬립있어서 순차 작동시켜놓은거 해체했습니다.

@109an94 109an94 requested review from Junh-b, bongj9 and caniro June 10, 2025 19:04
@caniro
Copy link
Contributor

caniro commented Jun 10, 2025

최근 변경사항이 롤백된 것 같습니다만 ㅋㅋㅋ 4시간 전 커밋 뭐지...
이따 깃 로그 같이 보면서 좀 더 보시죠!

@109an94 109an94 self-assigned this Jun 11, 2025
@109an94
Copy link
Contributor Author

109an94 commented Jun 11, 2025

최근 변경사항이 롤백된 것 같습니다만 ㅋㅋㅋ 4시간 전 커밋 뭐지... 이따 깃 로그 같이 보면서 좀 더 보시죠!

합치기 전으로 롤백해서 다시 병합후 올렸습니다!

@caniro
Copy link
Contributor

caniro commented Jun 11, 2025

최근 변경사항이 롤백된 것 같습니다만 ㅋㅋㅋ 4시간 전 커밋 뭐지... 이따 깃 로그 같이 보면서 좀 더 보시죠!
합치기 전으로 롤백해서 다시 병합후 올렸습니다!

@109an94 리플 인용하시면 작성하실 때 한 줄 띄우셔야 보는 사람 입장에서 내용이 제대로 보입니다 ㅋㅋ
bot 이외 코드 정상 병합된 것 확인했습니다~

Copy link
Contributor

@caniro caniro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

프로메테우스 메트릭 설정 적용 고생하셨습니다!
수식 알고리즘 논문 쓰셔도 되겠는데요 ㅋㅋㅋ

@investfuture-sonarqube1
Copy link

Failed Quality Gate failed

  • 13 New Issues (is greater than 0)
  • 0.00% Coverage on New Code (is less than 80.00%)

Project ID: CleanEngine_cleanengine-be_2b6f2f63-fa39-426c-b9c7-8aa127fd14d8

View in SonarQube

@109an94
Copy link
Contributor Author

109an94 commented Jun 11, 2025

@caniro 리뷰 감사합니다!

@109an94 109an94 merged commit 5d1defd into dev Jun 11, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants