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] 참여 인원이 없는 챌린지 deleted 처리 배치 개발 #311

Merged
merged 6 commits into from
May 1, 2023

Conversation

Youhoseong
Copy link
Collaborator

관련 Issue

작업 내용

참여 인원이 없는 챌린지 deleted 처리 배치 개발했어요.
범위 조회를 하고 싶어서 시간 파라미터를 넣고 싶은데요.
시간 흐름에 따라 deleted 처리되는 건 아니다 보니까 활용을 어캐해야될지 생각이 잘안나네요

다른 방법이더라도 데이터 스캔 범위를 어캐 해볼 수 있을지 의견 주시면 좋을거같아요!

@Youhoseong Youhoseong added the enhancement New feature or request label Apr 30, 2023
@Youhoseong Youhoseong requested a review from heoboseong7 April 30, 2023 16:56
@Youhoseong Youhoseong self-assigned this Apr 30, 2023
@github-actions
Copy link

테스트통과 🤖

@heoboseong7
Copy link
Collaborator

1일마다 스케쥴링을 걸어놓고 시간 정보를 활용한다면
challenge_participation의 status가 EXPIRED | CANCELED이고 updatedAt이 1일 이내인 것의 challenge들만 처리하는 방법은 어떤가요?

@Youhoseong
Copy link
Collaborator Author

@heoboseong7 이건 분 배치로 돌릴거같아요. 실시간 성이 보장되어야해서요!
챌린지 참가 상태가 변해도 challenge update는 안하지않나요? ㅜㅜ

@heoboseong7
Copy link
Collaborator

heoboseong7 commented May 1, 2023

@Youhoseong 그러면 분 단위로만 바꿔서 적용하면 될 것 같아요

챌린지 참가 상태가 변해도 challenge update는 안하지않나요? ㅜㅜ

-> 그래서 변경된 챌린지 참가를 찾은 다음, 연관된 챌린지를 찾아야 해요

@Youhoseong
Copy link
Collaborator Author

Youhoseong commented May 1, 2023

@heoboseong7 챌린지 참가 -> 챌린지랑 그거랑 엮인 모든 챌린지 참가 조회 이 구조?

@heoboseong7
Copy link
Collaborator

@Youhoseong 네 맞아요!

@Youhoseong
Copy link
Collaborator Author

@heoboseong7 적용했어요! 리뷰 고고

@github-actions
Copy link

github-actions bot commented May 1, 2023

테스트통과 🤖

Copy link
Collaborator

@heoboseong7 heoboseong7 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~~
젠킨스 스케쥴링도 잘 챙겨주세요 ㅎㅎ

Comment on lines 28 to 30
public List<Challenge> paginateWithParticipation(CursorPageable<Long> cPage) {
return challengeQueryRepository.paginateWithParticipation(cPage);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 메서드 관련해서 사용하지 않는 코드들은 삭제해도 될 것 같아요

@Youhoseong Youhoseong merged commit 35dd1cf into main May 1, 2023
@Youhoseong Youhoseong deleted the issue-#294 branch May 1, 2023 07:50
@github-actions
Copy link

github-actions bot commented May 1, 2023

테스트통과 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 참여자가 없는 챌린지 정보 삭제 배치 개발
2 participants