Skip to content

Conversation

@singsangssong
Copy link
Collaborator

…items

🔗 Related Issue

⌨️ What I did

특정 hot item의 읽기 트래픽을 여러 노드들로 분산시킬 수 있도록 Replica 기능을 구현합니다.
원본 키를 여러 복제키로 변환하여 무작위의 노드에 동일한 데이터를 분산시킵니다.

  • 저장(setReplicas): 지정된 갯수만큼 복제 키를 생성하고, 이를 여러 노드에 무작위로 분산시켜 저장합니다.
  • 조회(getFromReplica): 복제키들 중 하나를 랜덤하게 선택하여 조회합니다.
    • asyncGetFromReplica(): 비동기로 랜덤한 복제 키를 조회
    • getFromReplica(): 조회 성공까지 랜덤한 순서로 동기조회
  • ArcusClientReplicaTest
    • setReplicas()가 지정된 개수만큼 저장이 성공적으로 수행되는지 확인
    • getFromReplica()를 통해 저장된 값을 읽어오는지 확인

해당 PR이 머지됨에 따라 arcus-spring에도 replica 관련 메서드 적용한 PR올리도록 하겠습니다.

uhm0311
uhm0311 previously approved these changes Dec 5, 2025
@singsangssong
Copy link
Collaborator Author

arcus-spring에 특정키 중복저장에 대한 메서드를 적용하려면, arcus-java-client에 새로운 버전 릴리즈에 반영된 뒤에 추가되어야할것같은데, 이에 관해서 코멘트 부탁드립니다.

@uhm0311
Copy link
Collaborator

uhm0311 commented Dec 9, 2025

arcus-java-client가 먼저 릴리즈된 후에 arcus-spring 작업을 시작해도 어색하지 않을 것 같습니다.

Copy link
Collaborator

@oliviarla oliviarla left a comment

Choose a reason for hiding this comment

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

전체적인 틀은 approve 상태이고, 자잘한 구현 관점의 피드백입니다.

@oliviarla
Copy link
Collaborator

리뷰가 다 반영되었다면 재리뷰 요청해주세요.

@singsangssong
Copy link
Collaborator Author

@oliviarla
넵. 재리뷰 요청드립니다.

Copy link
Collaborator

@jhpark816 jhpark816 left a comment

Choose a reason for hiding this comment

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

리뷰 완료

@singsangssong singsangssong force-pushed the task/hotkey branch 3 times, most recently from 10c8bc2 to 775facc Compare December 18, 2025 07:45
@singsangssong singsangssong force-pushed the task/hotkey branch 2 times, most recently from c7741cb to fdb2dc7 Compare December 18, 2025 09:28
@jhpark816 jhpark816 requested a review from oliviarla December 19, 2025 00:15
Copy link
Collaborator

@oliviarla oliviarla left a comment

Choose a reason for hiding this comment

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

최종 리뷰입니다.

@oliviarla oliviarla requested a review from jhpark816 December 19, 2025 06:03
@jhpark816 jhpark816 merged commit 7aa2fce into naver:develop Dec 19, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants