Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: 푸쉬 알림 비동기 처리 #654

Merged
merged 12 commits into from
Nov 28, 2024
Merged

Conversation

ChooSeoyeon
Copy link
Member

📌 관련 이슈

close #642

✨ 작업 내용

  • 푸쉬 알림 비동기 처리

📚 기타

Comment on lines +14 to +21
int processors = Runtime.getRuntime().availableProcessors();
ThreadPoolTaskExecutor executor = new ThreadPoolTaskExecutor();
executor.setCorePoolSize(processors);
executor.setMaxPoolSize(processors * 2);
executor.setQueueCapacity(50);
executor.setThreadNamePrefix("AsyncExecutor-");
executor.initialize();
return executor;
Copy link
Contributor

Choose a reason for hiding this comment

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

application.yml 파일에 설정한 thread pool 설정이랑 곂치는지 궁금해요~

ThreadPoolExecutor는 비동기 쓰레드만 사용하는게 아닐것 같아서요 ㅎㅎ

Copy link
Contributor

@fromitive fromitive left a comment

Choose a reason for hiding this comment

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

에버, 도래 고생많았어요 ㅎㅎ lgtm

OfferingService 에서 ApplicationEventPublisher를 주입받고 이벤트를 수동을 발생시킨 뒤
발생한 시작지(source)값과 처리할 매개변수를 전달하는 모습이 제 눈에는 자연스러워 보였어요 😄

@fromitive fromitive merged commit ad77f1f into chongdae Nov 28, 2024
fromitive added a commit that referenced this pull request Nov 28, 2024
* feat: 비동기 처리 config 구현

* feat: 참여 로직 비동기 처리

* feat: 나머지 로직 비동기 처리

* test: 이벤트 발행 로직 테스트

* feat: 비동기 로직 예외 처리

* test: 테스트 깨지는 기준 날짜 변경

* chore: dev CI/CD 스크립트 임시 변경

* chore: dev CI/CD 스크립트 임시 변경

* chore: dev CI/CD 스크립트 임시 변경

* chore: dev CI/CD 스크립트 임시 변경

* chore: dev CI/CD 스크립트 원상복구

---------

Co-authored-by: SCY <helenason@naver.com>
Co-authored-by: fromitive <fromitive@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ 푸쉬 알림 비동기 처리
3 participants