Skip to content
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.

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

Feat: 이벤트 리스너가 로직을 비동기 처리하도록 변경한다. #127

Merged
merged 6 commits into from
May 26, 2024

Conversation

hseong3243
Copy link
Owner

@hseong3243 hseong3243 commented May 26, 2024

작업 내용

  • 이벤트 리스너가 비동기적으로 작동되도록 변경하였습니다.
  • 비동기 테스트를 위해 awaitility를 추가하였습니다.
  • 사용자가 직접적으로 사용할 수 없는 기능인 태그 자동 생성은 실패시 다시 호출합니다.
    • 처음에는 성공, 실패 여부를 기록하기 위해 별도의 엔티티를 추가하고 스케줄러를 이용하려고 하였으나, spring-event가 더 적절한 듯 하여 이를 이용하였습니다.
    • ApiException이 발생한 경우 60초, 120초 간격으로 3회 시도하며 실패 시 error 로그를 기록하고 예외를 던집니다.
  • 비동기 처리 실패 시 warn 로그를 기록하고 종료합니다.

@hseong3243 hseong3243 added the Feat 새로운 기능을 개발합니다. label May 26, 2024
@hseong3243 hseong3243 self-assigned this May 26, 2024
Copy link

Test Results

208 tests  +4   208 ✅ +4   5s ⏱️ -1s
 68 suites +3     0 💤 ±0 
 68 files   +3     0 ❌ ±0 

Results for commit 972a8ba. ± Comparison against base commit 6c52814.

@hseong3243 hseong3243 merged commit 2120dbd into main May 26, 2024
2 checks passed
@hseong3243 hseong3243 deleted the feat/#126-async-event branch May 26, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능을 개발합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: 이벤트 리스너가 로직을 비동기 처리하도록 변경한다.
1 participant