-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat/1 redis sortset queue #2
Conversation
대기열 서버가 spring application에 입장해도되는지 확인하는 과정은 없나요? |
@minjun3021 이미지를 보시면 (4)번 과정이 스프링애플리케이션이 직접 대기열 서버로 요청을 전송하도록 했습니다. Waiting => Running으로 옮기는 과정은 #3 에서 Node.js로 스케줄러를 만들 계획입니다. |
4번은 해당 티켓 번호가 유효한지 확인하는 작업아닌가요? 제가 말하는 거는 Waiting => Running으로 옮기는 과정에서 |
그렇게 현재 자원을 비교해서 대기열을 조정 하는건 진행하지않습니다. 스파이크 테스트를 통해서 DB가 1개의 자원에대해 1초동안에 안정적으로 응답 가능한 수가 100 리퀘스트면, 스케줄러가 1초에 50명씩 입장시킨다. 이렇게 구현 예정입니다 (inteval, move per inteval 설정) |
e3850a9
to
bbd8638
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
설계하신대로 잘 구현한 것 같습니다.
다만 동시접속자가 많지 않을때도 위와 같이 일관적으로 위와같은 모든 프로세스를 거치는것인가요?
넵 그부분이 아쉽긴하지만 동접자에따라 구분하는 기능 넣기는 구현 복잡도가 올라갈거같아 제외했습니다 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What is this PR?
Redis Sorted Set 기능을 이용해 Nodejs API 개발
Key Changes
Test Checklist