Skip to content

Refactor #89 : restClient timeout 설정 추가#91

Merged
seokhwan-an merged 2 commits intodevelopfrom
feature/refactor_restclient_time_out_setting
Apr 24, 2025
Merged

Refactor #89 : restClient timeout 설정 추가#91
seokhwan-an merged 2 commits intodevelopfrom
feature/refactor_restclient_time_out_setting

Conversation

@seokhwan-an
Copy link
Copy Markdown
Collaborator

@seokhwan-an seokhwan-an commented Apr 1, 2025

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명(이미지 첨부 가능)

해당 작업을 수행 배경

  • 외부 api의 연결과정이 지연되거나 혹은 데이터를 읽어오는 과정이 지연되는 경우 db connection이 문제가 될 수 있다고 판단했고 이를 직접 실험해보았습니다.
  • 40초의 연결지연이 발생하는 상황에서 100명의 사용자가 요청을 보낼 경우 아래와 같은 에러 발생
    error

작업 내용

  • 외부 api 통신에 이용되는 restClient timeout 설정 추가
  • 현재 timeout 설정 시간은 connection timeout, read timeout은 모두 5초로 설정했습니다 (https://whatap.io/bbs/board.php?bo_table=blog&wr_id=66 참고 + 제 경험을 바탕으로 추론했습니다.)

참고자료

💬 작업 시 고민사항

기능을 추가하거나 수정하는 상황에서 의문이 생긴 점이나 배운점 추가

고민사항

#️⃣ 연관된 이슈

연관된 이슈 번호를 모두 작성

@seokhwan-an seokhwan-an self-assigned this Apr 1, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2025

Test Results

148 tests   148 ✅  10s ⏱️
 51 suites    0 💤
 51 files      0 ❌

Results for commit 5a5c0fa.

♻️ This comment has been updated with latest results.

@seokhwan-an seokhwan-an merged commit e701a9d into develop Apr 24, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] restClient time out 설정 추가

1 participant