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

[Fix] #261 - O-KR 히스토리 조회 API 변경사항 반영 #262

Merged
merged 2 commits into from
Apr 8, 2024

Conversation

its-sky
Copy link
Member

@its-sky its-sky commented Apr 8, 2024

🚀PullRequest🚀

📟 관련 이슈

💻 작업 내용

O-KR 히스토리 조회 API 변경사항 반영

  • 데이터 스펙이 변경됨에 따라 year를 stream에 처리하는 로직이 제거되어 어느정도 성능이 빨라졌는지 궁금해서 JMeter 테스트 결과를 아래에첨부하였습니다~ 동시에 쓰레드 100개가 100번 요청하는 상황으로 부하테스트를 진행하였습니다.

아래 테스트 결과를 보시면 기존 방식과 변경된 방식에서 평균 API latency는 비슷하지만 95% 99% 즉 가장 최악의 API 수행 시간은 기존 방식이 오래 걸림을 확인할 수 있었습니다. 아마 stream 처리 때문에 처리될 쓰레드가 쌓이면서 처리시간이 늘어나지 않았을까 추측해봅니다.
그리고 대규모 건당 처리량이 엄청 드라마틱하게 늘어난 것은 아니지만 꽤나 늘어난 것을 볼 수 있습니다.

📝 리뷰 노트

[기존 API 구현 방법의 테스트]
스크린샷 2024-04-08 오후 9 49 08
스크린샷 2024-04-08 오후 9 49 23

[변경된 API 구현 방법의 테스트]
스크린샷 2024-04-08 오후 9 51 48
스크린샷 2024-04-08 오후 9 51 58
스크린샷 2024-04-08 오후 9 52 05

@its-sky its-sky requested a review from 0lynny April 8, 2024 12:58
@its-sky its-sky self-assigned this Apr 8, 2024
Copy link
Member

@0lynny 0lynny left a comment

Choose a reason for hiding this comment

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

부하테스트까지 수고 많으셨습니다 ~! 확실히 성능이 좋아진 것 같네요 !

@its-sky its-sky merged commit 044f341 into develop Apr 8, 2024
1 check passed
@its-sky its-sky deleted the feature/#261 branch April 8, 2024 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] O-KR 히스토리 조회 API 변경사항 반영
2 participants