Skip to content

Feat/trade core #130

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
caniro merged 6 commits into from
Jun 6, 2025
Merged

Feat/trade core #130

caniro merged 6 commits into from
Jun 6, 2025

Conversation

@caniro
Copy link
Contributor

@caniro caniro commented Jun 6, 2025 •
edited
Loading

✨ 작업내용

  • 체결 안정화(단일 스레드 동작, 재수정 필요)

🐞 이슈사항

이슈 번호 제목 상태
#129 [BE] 체결내역이 DB에 저장되지 않는 문제 ✅ 해결

⚠️ 특별사항

  • 체결 관련 히스토리
    • 6/4 배치 처리 -> 이벤트 핸들링 방식으로 전환
    • 6/5 @ASync 어노테이션을 통해 비동기 이벤트 방식으로 변경
      • 체결내역이 DB에 저장되지 않는 문제 발생
    • 6/6 @Async 어노테이션을 제거하여 동기 이벤트 방식으로 변경
      • 이벤트 핸들링 시 트랜잭션이 적용되지 않는 문제 발생 (파악 중)
      • 타 기능 테스트를 위한 안정화 우선 조치
        • Transactional의 Propagation을 REQUIRES_NEW 로 적용 시 신규 트랜잭션 생성되어 정상 처리 확인 완료
  • 추측이지만 Spring AOP에서 트랜잭션을 위한 Proxy 객체를 생성하는 범위와,
    이벤트 리스너 동작 방식에 대해 깊게 이해해야 해결 가능할 것 같습니다.

Junh-b and others added 5 commits June 5, 2025 10:49
@Junh-b
fix: 주문생성 임시 버그 수정
5aa19ff 
반환 예외 타입 변경으로 인한 핸들링 로직 변경
@Junh-b
fix: 이벤트 핸들러 별도 스레드 동작하도록 변경
0635421 
최근 반영사항으로 일시적으로 기존 싱글 스레드에서 처리되던 로직을 다시 별도 스레드 동작 방식으로 변경했습니다.
@caniro
refactor: 접근 제한자 및 리턴 타입 변경
0871699
@caniro
fix: 체결 멀티스레드 -> 단일스레드 변경(체결 안정화를 위한 임시 조치)
9d41687
@caniro
chore: 불필요 테스트 Disable 지정(추후 재작성 필요)
53ad317
@caniro caniro requested review from 109an94, Junh-b and bongj9 June 6, 2025 05:35
@caniro
Merge branch 'dev' into feat/trade-core
e2320d8 
# Conflicts:
#	src/main/java/com/cleanengine/coin/realitybot/service/OrderGenerateService.java
@caniro caniro force-pushed the feat/trade-core branch from 09d1499 to e2320d8 Compare June 6, 2025 05:43
109an94
109an94 approved these changes Jun 6, 2025
View reviewed changes
Copy link
Contributor

@109an94 109an94 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

investfuture-sonarqube1 bot commented Jun 6, 2025

Passed Quality Gate passed

Issues

Measures

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

View in SonarQube

@caniro caniro marked this pull request as ready for review June 6, 2025 05:45
@caniro
Copy link
Contributor Author

caniro commented Jun 6, 2025 •
edited
Loading

마침 딱 들어갔는데 풀리퀘 올라온게 있네요 문제 해결하시느라 고생하셨습니다

@109an94 미리 발견해주셔서 덕분에 일찍 조치했네요 감사합니다
늦게 알았으면 멘붕왔을 듯... ㅋㅋ

@caniro caniro self-assigned this Jun 6, 2025
@caniro caniro linked an issue Jun 6, 2025 that may be closed by this pull request
[BE] 체결내역이 DB에 저장되지 않는 문제 #129
Closed
@caniro caniro merged commit e17e715 into dev Jun 6, 2025
1 check passed
@caniro caniro mentioned this pull request Jun 7, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@109an94 109an94 109an94 approved these changes

@Junh-b Junh-b Awaiting requested review from Junh-b

@bongj9 bongj9 Awaiting requested review from bongj9

Assignees

@caniro caniro

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BE] 체결내역이 DB에 저장되지 않는 문제

4 participants

@caniro @109an94 @Junh-b