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

[#092] 매 00:00 분에 사용자 정보를 업데이트 하는 기능 구현 #93

Merged
merged 18 commits into from
Apr 22, 2024

Conversation

koomin1227
Copy link
Member

이슈

체크리스트

  • 00:00 분에 사용자 정보를 업데이트 하는 기능 구현
  • 트랜잭션 적용

고민한 내용

시간

  • 시간은 임의로 00:00 시로 해두었는데 나중에 다시 정하면 될 것 같습니다.

github update

  • 원해 update 메서드에서 한번에 모든 유저의 값을 업데이트 하도록 되어있었는데 종합 점수도 업데이트 해야했기에 하나의 유저의 값만 업데이트 하도록 수정하였습니다.

트랜잭션

  • 업데이트 하는 도중에 다른 커넥션에서 읽지 못하게 하려고 read commit 레벨로 트랜잭션을 설정 했습니다.

@koomin1227 koomin1227 requested a review from namewhat99 April 22, 2024 10:10
@koomin1227 koomin1227 self-assigned this Apr 22, 2024
await this.githubService.updateGithub(user.userId);
await this.totalService.updateTotal(user.userId);
}
this.logger.log(`모든 유저의 역량 업데이트 완료`);
Copy link
Contributor

Choose a reason for hiding this comment

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

확인했습니다~ 수고수고

@namewhat99 namewhat99 merged commit c9c6a9b into develop Apr 22, 2024
1 check passed
@koomin1227 koomin1227 deleted the feature/Batch-#92 branch April 22, 2024 14:35
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.

2 participants