[제안] id만 있는 BaseEntity 추가
EunjiShin opened this issue · 1 comments
EunjiShin commented
💁♀️ 제안 사항
- baseEntity를 두 개 관리해요.
- id, createdAt, updatedAt, deletedAt 이 있는 버전 (기존)
- id만 있는 버전 (신규)
👀 제안 이유
데이터 관리 목적으로 createdAt, updatedAt, deletedAt를 DB 공통 컨벤션으로 정했는데, 몇몇 테이블은 정말 저 필드들이 절대 쓰이지 않을 것 같아요. (e.g., 공감 테이블, 블록 해시태그 테이블 등.... )
불필요한 컬럼을 가지고 있는건 비용(메모리 등) 측면에서도 좋지 않고, 코드 복잡도도 올라가요.
따라서 필요에 따라 필드를 사용할 수 있도록 BaseEntity를 두 개 사용하면 어떨까 해요.
EunjiShin commented
잠깐 이거 쫌 바보이슈였다
어차피 id를 조작할 일이 없으니, 필요하다면 BaseEntity를 상속받는게 아니라 id필드 직접 만들어주면 될듯ㅋㅋㅠ 클로즈할게요~