-
Notifications
You must be signed in to change notification settings - Fork 0
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
database: change database schema #3
Comments
이슈 및 수정사항
ps. Report을 할 때 단순히 Challenge의 id만 받는 것이 아닌 무엇에 대한 Report을 받는지도 중요할 거 같습니다. |
@choiyeeun1010 아무래도 서비스 파악이 다 안끝난 상태에서 ERD 정리를 진행을 한 것이라 의도와 다른 부분이 있을 수 있습니다 관련 작업 진행하시면서 답변 남겨주시면 저도 참고해보겠습니다. |
넵 반영했습니다. 스키마(jaksimDB) 따로 생성해서 테이블들 추가 해놨습니당 다만, 추가로 삭제한 부분이 있는데, |
아직 db 설계에 관해서 토론이 끝난것이 아니니까 issues reopen 하겠습니다 |
A: memeberType table은 추후 만약 유저의 권한 등과 연관이 있을 거 같아 추가했습니다.
A: 이부분은 말씀해주신게 맞는 거 같습니다 :) |
기존 database의 schema상 정리가 필요해보여 현재 database의 문제점을 공유하고 수정하면 좋을 거 같습니다.
현재 schema는 과도할 정도로 Relational이 잡혀있습니다. 이는 유지보수면에서 좋지 못합니다(ex: 추후 쓸모가 없어진 database를 정리하지 못하는 경우가 발생할 수 있음.)
table, column등 전반적인 면에서 파악이 힘들어질 거 같습니다.
The text was updated successfully, but these errors were encountered: