Skip to content

Conversation

@junha6316
Copy link

@junha6316 junha6316 commented Apr 21, 2025

COS-3202

  • InMemoryCacheProvider를 singletone으로 추가했습니다.
  • getPackageHistory, getDeployment 에 캐싱을 추가했습니다.
  • 관련된 테스트 코드를 작성했습니다.

@junha6316 junha6316 self-assigned this Apr 21, 2025
@junha6316 junha6316 added the enhancement New feature or request label Apr 21, 2025
@junha6316 junha6316 requested a review from mu29 April 21, 2025 06:24
@junha6316 junha6316 marked this pull request as ready for review April 21, 2025 06:24
Copy link
Collaborator

@mu29 mu29 left a comment

Choose a reason for hiding this comment

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

👍
캐시 크기가 얼마나 늘어날까요? 요청이 잦아서 워커 인스턴스는 계속 재활용될 것 같고, 혹시 계속 증가한다면 LRU가 필요할 수도요 ㅎㅎ https://github.com/isaacs/node-lru-cache

당장은 괜찮을 것 같긴 합니당

@junha6316
Copy link
Author

@mu29 좀 돌려 봐야 알거 같아요! 근데 저희 앱 밖에 없고 제일 큰게 길이가 200정도 되는 array이라 말씀대로 지금 당장은 괜찮을거 같습니다!

@junha6316 junha6316 merged commit 889000c into main Apr 22, 2025
2 checks passed
@junha6316
Copy link
Author

@mu29 r2 권한때문에 배포도 안되네요..ㅎㅎ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants