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.

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

[fix] SSE 문제 해결 #58

Merged
merged 14 commits into from
Dec 5, 2023
Merged

[fix] SSE 문제 해결 #58

merged 14 commits into from
Dec 5, 2023

Conversation

yonghwankim-dev
Copy link
Member

구현한 것

  • 쓰레드풀의 크기를 1로 변경하여 종목 가격 및 종목 종가 갱신시 open api 서버로부터 요청 횟수 초과 응답을 받지 않도록 변경하였습니다.
  • SSE 테스크 부분에 Exception 예외를 추가하여 모든 예외에 대하여 로깅할 수 있도록 변경하였습니다.
  • KisService의 스케줄링 메소드들을 코드 정리하였습니다.
  • dev-cicd 브랜치를 통하여 개발 서버에 배포하여 sse 응답이 잘 오는 것을 확인하였습니다.

@yonghwankim-dev yonghwankim-dev added the bug Something isn't working label Dec 5, 2023
@yonghwankim-dev yonghwankim-dev self-assigned this Dec 5, 2023
@yonghwankim-dev yonghwankim-dev linked an issue Dec 5, 2023 that may be closed by this pull request
[bug] 포트폴리오 상세 조회 SSE 데이터 응답 문제 #57
Closed
@yonghwankim-dev yonghwankim-dev merged commit d69a4cb into dev Dec 5, 2023
@yonghwankim-dev yonghwankim-dev deleted the fix/#57-sse branch December 5, 2023 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yhpark95 yhpark95 yhpark95 approved these changes

@yein-lee yein-lee yein-lee approved these changes

Assignees

@yonghwankim-dev yonghwankim-dev

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[bug] 포트폴리오 상세 조회 SSE 데이터 응답 문제
3 participants
@yonghwankim-dev @yein-lee @yhpark95