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

[feat] 투표 마감 기능 구체화 #57

Closed
3 tasks done
NaMinhyeok opened this issue Oct 2, 2024 · 0 comments · Fixed by #62
Closed
3 tasks done

[feat] 투표 마감 기능 구체화 #57

NaMinhyeok opened this issue Oct 2, 2024 · 0 comments · Fixed by #62
Assignees
Labels
✨Feature 새로운 기능 추가 나민혁

Comments

@NaMinhyeok
Copy link
Collaborator

NaMinhyeok commented Oct 2, 2024

✨ 구현 할 기능

  • 투표 마감 조건에 따른 스케줄러에서 task 실행시키기
  • 종료 할 때 투표를 찾을 때 예외처리
  • 시스템이 종료될 때를 대비해서 첫 실행시 스케줄러에 task 추가

구현 방법 생각

Runnable로 만들어서 조건이 맞으면 Runnable이 실행되게 하면 될 것 같다는 생각
ScheduledFuture를 이용하면 취소 할 수는 있다고 한다. 전체 취소인지,특정 task 취소인지는 알아볼 것

initialize는 상태를 찾아서 이미 종료된 것 제외하고 가져와서 endAt을 확인해서 스케줄러에 추가하면 될 것으로 보임

📕 레퍼런스

@NaMinhyeok NaMinhyeok added ✨Feature 새로운 기능 추가 나민혁 labels Oct 2, 2024
@NaMinhyeok NaMinhyeok added this to the 2주차 마일스톤 milestone Oct 2, 2024
@NaMinhyeok NaMinhyeok self-assigned this Oct 2, 2024
NaMinhyeok added a commit that referenced this issue Oct 6, 2024
- 소프트딜리트를 적용한 findBy 구현
- redis config 수정
- 엔티티 수정사항에 맞춰 테스트 수정
- 기타 Displayname안들어있는거 수정
NaMinhyeok added a commit that referenced this issue Oct 6, 2024
- 사용자 입력을 통한 마감 구현
- 투표 상태 응답에 추가
- concurrenthashmap으로 인메모리상에 있는 문제 존재
- 예상치 못하게 종료되어도 PostConstruct로 다시 스케쥴링
NaMinhyeok added a commit that referenced this issue Oct 6, 2024
- 테스트 깨지는 것은 PR merge된 이후에 한번에 수정하도록 하겠습니다
@NaMinhyeok NaMinhyeok linked a pull request Oct 6, 2024 that will close this issue
3 tasks
NaMinhyeok added a commit that referenced this issue Oct 7, 2024
NaMinhyeok added a commit that referenced this issue Oct 7, 2024
…능-구체화

[Feat/#57] 투표 마감 기능 구체화
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature 새로운 기능 추가 나민혁
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant