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

[BE] feat#333 updatePos socket batch 처리 #334

Merged
merged 4 commits into from
Dec 23, 2024

Conversation

DongHoonYu96
Copy link
Collaborator

@DongHoonYu96 DongHoonYu96 commented Dec 23, 2024

➕ 이슈 번호


🔎 작업 내용

이 Pull Request는 BE/src/game/redis/subscribers/player.subscriber.ts 파일의 PlayerSubscriber 클래스의 변경사항을 포함하며, 플레이어 위치 업데이트 처리의 최적화에 중점을 두었습니다. 가장 중요한 변경사항은 위치 업데이트를 저장할 새로운 맵 추가, 정기적인 간격으로 위치 업데이트를 배치 처리하고 전송하는 기능, 그리고 플레이어 상태에 따른 위치 업데이트 처리 방식의 리팩토링을 포함합니다.

플레이어 위치 업데이트 최적화:

  • 플레이어의 위치 업데이트를 저장하기 위한 positionUpdatespositionUpdatesForDead 맵을 추가했습니다.
  • positionUpdates 맵의 위치 업데이트를 100밀리초마다 배치 처리하고 전송하는 setInterval 함수를 구현했습니다.
  • 생존한 플레이어의 위치 업데이트 처리를 즉시 전송하는 대신 positionUpdates 맵에 저장하도록 리팩토링했습니다.

🖼 참고 이미지


🎯 리뷰 요구사항 (선택)

  • 특별히 봐줬으면 하는 부분이 있다면 적어주세요

✅ Check List

  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

@Copilot Copilot bot review requested due to automatic review settings December 23, 2024 05:58

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

@DongHoonYu96 DongHoonYu96 changed the base branch from main to dev-be December 23, 2024 05:58
@DongHoonYu96 DongHoonYu96 merged commit 6c0957e into boostcampwm-2024:dev-be Dec 23, 2024
@DongHoonYu96 DongHoonYu96 deleted the feture-be-#333 branch December 23, 2024 08:59
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.

4 participants