Skip to content

Conversation

@109an94
Copy link
Contributor

@109an94 109an94 commented Jun 11, 2025

✨ 작업내용

  • 스케줄러 두번씩 작동 수정 + 7ab9d57
  • unitpricerefresher 호출 순서 수정 + 7ab9d57

🐞 이슈사항

이슈 번호 제목 상태
#000 예시 이슈 ✅ 해결

⚠️ 특별사항

  • 처음에 application run 실행 시 unitprice를 계산하도록 했다가 schedulerconfig에서 schedulerregistrar 작동이 더 빨라서 해당 매서드로 unitprice를 먼저 계산하도록하여 NPE문제를 해결했습니다.

@109an94 109an94 requested review from Junh-b, bongj9 and caniro June 11, 2025 09:11
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.

스케줄링 로직 수정 고생하셨습니다!
소중한 테스트들이..ㅠㅠ

Copy link
Contributor

@Junh-b Junh-b left a comment

Choose a reason for hiding this comment

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

수정내용 확인했습니다
저도 문제 내용을 처음 들었을 때 Order를 수정하는 방식을 먼저 떠올렸었는데, 초기 scheduling 설정과 unitprice의 설정은 밀접한 관련이 있으니, 이렇게 한쪽에서 호출하는 방식으로 구현하신것이 더 괜찮게 보이네요.

@investfuture-sonarqube1
Copy link

Failed Quality Gate failed

  • 1 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 109an94 merged commit 748d485 into dev Jun 11, 2025
1 check passed
@109an94
Copy link
Contributor Author

109an94 commented Jun 11, 2025

수정내용 확인했습니다 저도 문제 내용을 처음 들었을 때 Order를 수정하는 방식을 먼저 떠올렸었는데, 초기 scheduling 설정과 unitprice의 설정은 밀접한 관련이 있으니, 이렇게 한쪽에서 호출하는 방식으로 구현하신것이 더 괜찮게 보이네요.

감사합니다 또 h2환경에선 똑같은 문제가 발생해서 둘이 빌드되는 과정에서 뭐가 다른지 이참에 학습해봐야겠어요

@109an94 109an94 self-assigned this Jun 13, 2025
@109an94 109an94 mentioned this pull request Jun 14, 2025
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.

4 participants