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

처음 회원가입하는 로직을 비관적락 으로 수정 #131

Merged
merged 18 commits into from
Aug 26, 2024

Conversation

ysndy
Copy link
Collaborator

@ysndy ysndy commented Aug 25, 2024

연관된 이슈

https://watermelon-clap.atlassian.net/jira/software/projects/WB/boards/2?selectedIssue=WB-286

작업 내용

처음 회원가입하는 로직에 대해 동시성 보장을 위해 비관적락을 적용했습니다.

  • READ_COMMITTED 를 적용해서 롤백되지 않는 경우에서의 원자성 보장에 성공했지만, 롤백되는 경우를 고려하지 못해서 다시 낙관적 락으로 수정했습니다.

참고 자료

https://www.notion.so/bside/BE-47212a8616b747019dd64afd32ff4a60

Copy link

github-actions bot commented Aug 25, 2024

Unit Test Results

  36 files    36 suites   12s ⏱️
144 tests 144 ✔️ 0 💤 0
145 runs  145 ✔️ 0 💤 0

Results for commit e103efd.

♻️ This comment has been updated with latest results.

Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

1 similar comment
Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

3 similar comments
Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

1 similar comment
Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

1 similar comment
Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

1 similar comment
Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

1 similar comment
Copy link

테스트 커버리지 측정

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

@ysndy ysndy merged commit 1b79f88 into dev Aug 26, 2024
3 checks passed
ysndy added a commit that referenced this pull request Aug 26, 2024
* fix[api-server]: 링크, 응모자, 응모자-파츠 엔티티 생성전략 IDENTITY로 수정

* fix[api-server]: 로터리 서비스에서 불필요한 isExist 메소드 제거

* refactor[api-server]: 링크 서비스에서 불필요한 addLinkViewCount 메소드 제거

* feat[api-server]: LotteryApplier 엔티티의 변화를 로깅하는 리스너 추가

* fix[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* feat[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* fix[api-server]: 처음 로그인 로직을 트랜잭션으로 수정

* fix[api-server]: 처음 로그인 로직 관련 테스트 수정

* fix[api-server]: 링크 조회수 증가 로직 수정

* fix[api-server]: 처음 로그인 로직에 낙관적 락 적용

* fix[api-server]: BaseEntity에 ToSting 제거

* fix[api-server]: 첫 번째 회원가입 로직에 비관적 락 적용

* fix[api-server]: 첫 번째 회원가입 로직에 낙관적 락 적용

* fix[api-server]: 선착순 이벤트 스케줄러 레이트 500으로 설정

* fix[api-server]: toString 제거

* fix[api-server]: 엔티티리스너 제거
ysndy added a commit that referenced this pull request Aug 26, 2024
* 처음 회원가입하는 로직을 비관적락 으로 수정 (#131)

* fix[api-server]: 링크, 응모자, 응모자-파츠 엔티티 생성전략 IDENTITY로 수정

* fix[api-server]: 로터리 서비스에서 불필요한 isExist 메소드 제거

* refactor[api-server]: 링크 서비스에서 불필요한 addLinkViewCount 메소드 제거

* feat[api-server]: LotteryApplier 엔티티의 변화를 로깅하는 리스너 추가

* fix[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* feat[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* fix[api-server]: 처음 로그인 로직을 트랜잭션으로 수정

* fix[api-server]: 처음 로그인 로직 관련 테스트 수정

* fix[api-server]: 링크 조회수 증가 로직 수정

* fix[api-server]: 처음 로그인 로직에 낙관적 락 적용

* fix[api-server]: BaseEntity에 ToSting 제거

* fix[api-server]: 첫 번째 회원가입 로직에 비관적 락 적용

* fix[api-server]: 첫 번째 회원가입 로직에 낙관적 락 적용

* fix[api-server]: 선착순 이벤트 스케줄러 레이트 500으로 설정

* fix[api-server]: toString 제거

* fix[api-server]: 엔티티리스너 제거

* deploy 브랜치 분리 및 CI/CD 추가 (#132)

* fix[api-server]: 링크, 응모자, 응모자-파츠 엔티티 생성전략 IDENTITY로 수정

* fix[api-server]: 로터리 서비스에서 불필요한 isExist 메소드 제거

* refactor[api-server]: 링크 서비스에서 불필요한 addLinkViewCount 메소드 제거

* feat[api-server]: LotteryApplier 엔티티의 변화를 로깅하는 리스너 추가

* fix[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* feat[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* fix[api-server]: 처음 로그인 로직을 트랜잭션으로 수정

* fix[api-server]: 처음 로그인 로직 관련 테스트 수정

* fix[api-server]: 링크 조회수 증가 로직 수정

* fix[api-server]: 처음 로그인 로직에 낙관적 락 적용

* fix[api-server]: BaseEntity에 ToSting 제거

* fix[api-server]: 첫 번째 회원가입 로직에 비관적 락 적용

* fix[api-server]: 첫 번째 회원가입 로직에 낙관적 락 적용

* fix[api-server]: 선착순 이벤트 스케줄러 레이트 500으로 설정

* fix[api-server]: toString 제거

* fix[api-server]: 엔티티리스너 제거

* feat[api-server]: deploy 배포 CI/CD 추가
ysndy added a commit that referenced this pull request Aug 26, 2024
* 처음 회원가입하는 로직을 비관적락 으로 수정 (#131)

* fix[api-server]: 링크, 응모자, 응모자-파츠 엔티티 생성전략 IDENTITY로 수정

* fix[api-server]: 로터리 서비스에서 불필요한 isExist 메소드 제거

* refactor[api-server]: 링크 서비스에서 불필요한 addLinkViewCount 메소드 제거

* feat[api-server]: LotteryApplier 엔티티의 변화를 로깅하는 리스너 추가

* fix[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* feat[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* fix[api-server]: 처음 로그인 로직을 트랜잭션으로 수정

* fix[api-server]: 처음 로그인 로직 관련 테스트 수정

* fix[api-server]: 링크 조회수 증가 로직 수정

* fix[api-server]: 처음 로그인 로직에 낙관적 락 적용

* fix[api-server]: BaseEntity에 ToSting 제거

* fix[api-server]: 첫 번째 회원가입 로직에 비관적 락 적용

* fix[api-server]: 첫 번째 회원가입 로직에 낙관적 락 적용

* fix[api-server]: 선착순 이벤트 스케줄러 레이트 500으로 설정

* fix[api-server]: toString 제거

* fix[api-server]: 엔티티리스너 제거

* deploy 브랜치 분리 및 CI/CD 추가 (#132)

* fix[api-server]: 링크, 응모자, 응모자-파츠 엔티티 생성전략 IDENTITY로 수정

* fix[api-server]: 로터리 서비스에서 불필요한 isExist 메소드 제거

* refactor[api-server]: 링크 서비스에서 불필요한 addLinkViewCount 메소드 제거

* feat[api-server]: LotteryApplier 엔티티의 변화를 로깅하는 리스너 추가

* fix[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* feat[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* fix[api-server]: 처음 로그인 로직을 트랜잭션으로 수정

* fix[api-server]: 처음 로그인 로직 관련 테스트 수정

* fix[api-server]: 링크 조회수 증가 로직 수정

* fix[api-server]: 처음 로그인 로직에 낙관적 락 적용

* fix[api-server]: BaseEntity에 ToSting 제거

* fix[api-server]: 첫 번째 회원가입 로직에 비관적 락 적용

* fix[api-server]: 첫 번째 회원가입 로직에 낙관적 락 적용

* fix[api-server]: 선착순 이벤트 스케줄러 레이트 500으로 설정

* fix[api-server]: toString 제거

* fix[api-server]: 엔티티리스너 제거

* feat[api-server]: deploy 배포 CI/CD 추가

* deploy 배포 CI/CD 테스트 (#135)

* fix[api-server]: 링크, 응모자, 응모자-파츠 엔티티 생성전략 IDENTITY로 수정

* fix[api-server]: 로터리 서비스에서 불필요한 isExist 메소드 제거

* refactor[api-server]: 링크 서비스에서 불필요한 addLinkViewCount 메소드 제거

* feat[api-server]: LotteryApplier 엔티티의 변화를 로깅하는 리스너 추가

* fix[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* feat[api-server]: 엔티티 변경사항 로깅을 BaseEntity 에 공통으로 적용

* fix[api-server]: 처음 로그인 로직을 트랜잭션으로 수정

* fix[api-server]: 처음 로그인 로직 관련 테스트 수정

* fix[api-server]: 링크 조회수 증가 로직 수정

* fix[api-server]: 처음 로그인 로직에 낙관적 락 적용

* fix[api-server]: BaseEntity에 ToSting 제거

* fix[api-server]: 첫 번째 회원가입 로직에 비관적 락 적용

* fix[api-server]: 첫 번째 회원가입 로직에 낙관적 락 적용

* fix[api-server]: 선착순 이벤트 스케줄러 레이트 500으로 설정

* fix[api-server]: toString 제거

* fix[api-server]: 엔티티리스너 제거

* feat[api-server]: deploy 배포 CI/CD 추가

* style[api-server]: deploy 배포 CI/CD 테스트
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.

1 participant