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

[Feat] BaseEntity 추가, 지연로딩 추가 #22

Merged
merged 2 commits into from
Jan 5, 2025

Conversation

david-parkk
Copy link
Member

✏️ 작업 개요

⛳ 작업 분류

  • BaseEntity 추가
  • 지연로딩 추가

🔨 작업 상세 내용

  1. BaseEntity 사용을 제안하고자 합니다.
  2. created at, modified at 컬럼을 AuditingEntityListener 를 통해 삽입, 수정되는 경우에 해당 시간으로 저장하도록 합니다.
  3. 기존에 있는 모든 엔티티에 적용하였습니다.
  4. 다대일 연관관계에 대해서 지연로딩을 적용하였습니다. (의도했던 것은 아니고 깜박했습니다..)

💡 생각해볼 문제

  • 개발중에 이렇게 추가적으로 PR이 필요한 경우 issue를 파서 브랜치에 작업을 해야할지, isssue없이 브랜치로만 작업할지, 기존에 있는 브랜치로 작업하는게 좋은지 정하면 좋겠어요(이번에는 운좋게 적합한 브랜치가 이미 있었어요)

Copy link

github-actions bot commented Jan 3, 2025

Test Results

1 tests   1 ✅  0s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit 74332c8.

Copy link
Contributor

@You-Hyuk You-Hyuk left a comment

Choose a reason for hiding this comment

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

JPA Auditing 이라는 것을 처음 알게됐는데, 엔티티에서 발생한 이벤트를 추적하고 기록하는 기능을 통해 엔티티들의 공통된 필드인 createdAt, modifiedAt을 하나의 추상 클래스로 관리할 수 있다는 점이 굉장히 좋은 것 같아요!

추가적인 PR에 대해서는 저도 크지 않은 단위의 변경만을 필요로 할 때에도 Issue를 등록한 뒤 해당 브랜치에서 작업 후 PR을 날리는 것이 맞는 방식인지에 대한 고민이 있었습니다. 개인적인 생각으로는

본인이 생각하기에 서비스 로직의 큰 변화가 있는 것이 아니고 적합한 브랜치가 존재한다면 기존 브랜치 사용

서비스 로직의 변화가 크거나 새로운 것일 경우 Issue를 등록하여 해당 브랜치에서 작업

이와 같은 방식으로 생각해보았는데, 이렇게 진행해도 괜찮을지 한 번 확인해주시면 감사하겠습니당

@david-parkk
Copy link
Member Author

JPA Auditing 이라는 것을 처음 알게됐는데, 엔티티에서 발생한 이벤트를 추적하고 기록하는 기능을 통해 엔티티들의 공통된 필드인 createdAt, modifiedAt을 하나의 추상 클래스로 관리할 수 있다는 점이 굉장히 좋은 것 같아요!

추가적인 PR에 대해서는 저도 크지 않은 단위의 변경만을 필요로 할 때에도 Issue를 등록한 뒤 해당 브랜치에서 작업 후 PR을 날리는 것이 맞는 방식인지에 대한 고민이 있었습니다. 개인적인 생각으로는

본인이 생각하기에 서비스 로직의 큰 변화가 있는 것이 아니고 적합한 브랜치가 존재한다면 기존 브랜치 사용

서비스 로직의 변화가 크거나 새로운 것일 경우 Issue를 등록하여 해당 브랜치에서 작업

이와 같은 방식으로 생각해보았는데, 이렇게 진행해도 괜찮을지 한 번 확인해주시면 감사하겠습니당

좋습니다!

@david-parkk david-parkk merged commit fd5fabe into develop Jan 5, 2025
5 checks passed
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