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

#240 [feat] jpql 로직 querydsl로 이전 #241

Merged
merged 7 commits into from
Dec 13, 2023
Merged

#240 [feat] jpql 로직 querydsl로 이전 #241

merged 7 commits into from
Dec 13, 2023

Conversation

KWY0218
Copy link
Member

@KWY0218 KWY0218 commented Dec 12, 2023

✒️ 관련 이슈번호

Key Changes 🔑

  1. 내용
    • jpql 로직 querydsl로 이전

To Reviewers 📢

  • JPQL 대신 JPA ORM을 사용하도록 수정했습니다.

  • JPQL을 통해 구현했던 아래 2개 로직을 querydsl로 수정했습니다.

    1. Update 벌크 연산
      • qeury dsl로 인해 연산 이후 영속성 컨텍스트가 초기화 되지 않아서, meeting 수정 사항을 바로 flush 하지 않아도 됩니다.
    2. fetch join 연산
  • 배포 서버가 닫혀 있어서 code deploy 부분은 주석처리 했습니다.

@KWY0218 KWY0218 requested a review from sohyundoh December 12, 2023 12:37
@KWY0218 KWY0218 self-assigned this Dec 12, 2023
Copy link
Member

@sohyundoh sohyundoh left a comment

Choose a reason for hiding this comment

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

작업하느라 고생하셨습니다 :>
레포지토리가 많아져서 패키지 안으로 넣으신 거겠죠? 이런걸 보니 확실히 DDD로 도메인 단위로 패키지 나누는 것도 좋은 것 같네요.

@KWY0218 KWY0218 merged commit 8203b9f into develop Dec 13, 2023
1 check passed
@KWY0218 KWY0218 deleted the feat/#240 branch December 13, 2023 02:09
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.

[fix] 벌크성 update 로직 query dsl로 마이그레이션
2 participants