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

[BE] feature : transaction 적용 및 에러 fix #182

Merged
merged 5 commits into from
Dec 4, 2023

Conversation

HKLeeeee
Copy link
Collaborator

@HKLeeeee HKLeeeee commented Dec 4, 2023

PR 설명

  • transaction 적용 및 에러 fix

✅ 완료한 기능 명세

  • 쓰기 작업이 일어나는 경우만 transaction을 추가해주었습니다.
  • 코드 실행 시 임시 파일 생성 이름 : Date.now() - > uuid로 변경 , 여러 프로세스에서 동시 실행시 같은 이름에 파일이 생성, 삭제되어 오류남

📸 스크린샷

고민과 해결과정

  • 현재는 트랜잭션 코드 AOP 처리가 되어있지 않으나, 추후 service 레벨에 적용할 수 있는 트랜잭션 데코레이터를 생성해서 적용하는 것이 목표입니다.
    : AsyncLocalStorage 또는 cls-hooked 를 이용해서 http 요청 당 전역으로 이용할 수 있는 namespace를 만들어 쿼리러너를 이용하고 싶음.
    : 실패시 대안 1. controller 레벨에서 인터셉터, 데코레이터 활용. 2. 이미 만들어져있는 패키지 이용

timestampe -> uuid
다수 프로세서로 앱 동작시, 동시 요청이 들어오면 같은 이름에 파일이 생기는 경우 존재
@HKLeeeee HKLeeeee self-assigned this Dec 4, 2023
@HKLeeeee HKLeeeee added feature 기능 구현 BackEnd 백엔드 관련 fix 오류 수정 labels Dec 4, 2023
@Gseungmin
Copy link
Member

LGTM!

@Gseungmin Gseungmin merged commit f9e6f51 into boostcampwm2023:dev Dec 4, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BackEnd 백엔드 관련 feature 기능 구현 fix 오류 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants