Skip to content

FIX: Skip if nodeSet return null when already removed - #1122

Merged
jhpark816 merged 1 commit into
naver:developfrom
oliviarla:develop
Oct 14, 2026
Merged

jhpark816 merged 1 commit into
naver:developfrom
oliviarla:develop

Conversation

@oliviarla

@oliviarla oliviarla commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

🔗 Related Issue

⌨️ What I did

    1. 하나의 노드에서 160개 hash point 만들 때 동일한 hash point가 중복해서 나타나면서 2. 다른 노드에는 해당 hash point가 없고 3. 해당 노드가 클러스터에서 제외되는 경우 nodeSet.remove 메서드 호출 시 NullPointerException 발생하여 IO Thread가 종료되는 현상이 발생했습니다.
  • hash point 값을 가지는 nodeSet을 조회하고, 해당 nodeSet에서 현재 node를 제거했는데 다음 루프에서 또다시 동일한 hash point 값을 가지는 nodeSet을 조회했을 때에는 이미 ketamaNodes에서 nodeSet entry 자체가 제거된 상태가 됩니다.
  • 이 경우에 entry가 없어 null이 반환되면 continue 처리하여 중복 hash point가 있더라도 NullPointerException 발생하지 않도록 수정하였습니다.

@f1v3-dev f1v3-dev left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

removeHashOfAlter() 메서드도 수정되었으니, 중복 해시 포인트가 있는 노드를 updateAlter()로 제거하는 테스트도 추가하면 좋을 것 같습니다.

@oliviarla

Copy link
Copy Markdown
Collaborator Author

테스트 추가했습니다..!

@jhpark816
jhpark816 merged commit c58a9de into naver:develop Oct 14, 2026
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.

3 participants