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.

Sign up for GitHub

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

Jump to bottom

[BE] PATCH /post/:id 트랜잭션 개선, POST /post 트랜잭션 적용 #195

Merged
merged 2 commits into from
Nov 30, 2023
Merged

[BE] PATCH /post/:id 트랜잭션 개선, POST /post 트랜잭션 적용 #195

merged 2 commits into from
Nov 30, 2023

Conversation

qkrwogk
Copy link
Collaborator

@qkrwogk qkrwogk commented Nov 30, 2023 •
edited
Loading

📎 이슈번호

📃 변경사항

  • PATCH /post/:id 트랜잭션 개선
    • 계획
    • 구현
    • 동작 화면
  • POST /post 트랜잭션 적용
    • 구현
    • 동작 화면

🫨 고민한 부분

개발기록 참고

📌 중점적으로 볼 부분

🎇 동작 화면

PATCH /post/:id 트랜잭션 개선

스크린샷 2023-11-30 오후 2 15 39 스크린샷 2023-11-30 오후 2 16 15

이제 Image 삭제 및 새로 삽입하는 로직, 보드 수정 로직이 모두 하나의 트랜잭션에 들어간다. 성공!

POST /post 트랜잭션 적용

스크린샷 2023-11-30 오후 2 14 36
  • before
스크린샷 2023-11-30 오후 1 23 21
  • after
스크린샷 2023-11-30 오후 1 59 31

이제 Image와 Board 레코드의 생성 로직이 모두 한 트랜잭션에 들어간다. 성공!

💫 기타사항

@qkrwogk
⚡️ all updateBoard logic into one transaction
7ad121f 
- uploadFile에서 image 레코드 생성 로직 분리
- image entity에 etag 컬럼 추가 (valid한 이미지 업로드 보장)
- 트랜잭션 생성된 queryRunner에서 image 레코드 생성 로직 실행
@qkrwogk
⚡️ use transaction on createBoard
a16efe4 
- 트랜잭션 생성된 queryRunner에서 image 레코드 생성 및 board 레코드 생성 로직 실행
- 에러 발생 시 500 에러 처리 로직 추가
@qkrwogk qkrwogk added ⚡️ Enhancement 성능 개선 BE 백엔드 labels Nov 30, 2023
@qkrwogk qkrwogk added this to the Week 4 milestone Nov 30, 2023
@qkrwogk qkrwogk requested a review from SongJSeop November 30, 2023 05:36
@qkrwogk qkrwogk self-assigned this Nov 30, 2023
SongJSeop
SongJSeop approved these changes Nov 30, 2023
View reviewed changes
Copy link
Collaborator

@SongJSeop SongJSeop left a comment

Choose a reason for hiding this comment

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

좋습니다 ㅎㅎ

@SongJSeop SongJSeop merged commit 850e72d into boostcampwm2023:be-develop Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SongJSeop SongJSeop SongJSeop approved these changes

Assignees

@qkrwogk qkrwogk

Labels
BE 백엔드 ⚡️ Enhancement 성능 개선
Projects
None yet
Milestone
Week 4
Development

Successfully merging this pull request may close these issues.
2 participants
@qkrwogk @SongJSeop