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

Refactor/미사용 DB table 제거 #335

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

tmdgus1715
Copy link
Contributor

@tmdgus1715 tmdgus1715 commented Feb 4, 2024

▶ Request

Content

as-is

데이터베이스 테이블의 활용도 조사 결과 미사용중인 테이블이 존재하는 것을 확인 함.

to-be

기존 스테이지 서버에서 db를 /srv/old-schema 경로에 백업 후
미사용중인 DB테이블들을 제거한다.

✅ Check List

  • 의도치 않은 변경이 일어나지 않았는지.
    • 실수로 라이브러리(pom.xml) 변경이 일어나지 않았는지
    • 병합시 컴파일 & 런타임 에러가 발생하지 않는지
    • 기존 클라이언트와의 호환성 고려가 잘 이루어졌는지
  • 작성한 코드가 프로젝트에 반영됨을 명심하였는지
    • 타인도 알아보고 변경할 수 있는 코드를 작성하였는지
    • 코드 & 커밋 컨벤션을 준수하였는지
    • (필요한) 문서화가 진행되었는지
  • (기능 추가의 경우) 클라이언트의 입장에 대한 충분한 고려가 이루어졌는지
    • 클라이언트 측과 협의가 된 내용인 경우
    • 클라이언트의 요구사항을 잘 반영하는지
    • API 문서에 논리적인 오류 & 가시성이 떨어지는 내용이 없는지

📸 API Document ScreenShot

image -db backup-

🧪 Test

  • 로컬환경에서 현재 스테이지 dump파일로 테스트 수행

@tmdgus1715 tmdgus1715 changed the base branch from production to develop February 4, 2024 10:47
Copy link
Contributor

@Invidam Invidam left a comment

Choose a reason for hiding this comment

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

테이블 지우는 건 강조해도 모자를 만큼 위험한 작업이라 생각돼서

  1. 지우지 않는다.
  2. 확실하게 필요없는 테이블만 지운다.
  3. 여러 명이서 최소 N번이상 교차 검증한다.

중에 하는 건 어떨까?

+) 미사용 테이블을 조사한 방법이 있을 거라고 생각하는데 사용하고 있음에도 - 미사용으로 분류된 테이블이 있었다고 판단돼서 미사용이 정확히 어떻게 정의된건지, 조사한 방식에 허점이 없었는지 에 대해서도 다시 생각해봐야할 것 같아..

`koin`.`lost_items`,
`koin`.`lost_item_view_logs`,
`koin`.`migrations`,
`koin`.`notice_articles`,
Copy link
Contributor

@Invidam Invidam Feb 4, 2024

Choose a reason for hiding this comment

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

koin.notice_articles,

https://github.com/BCSDLab/KOIN_BATCH/blob/develop/crawling/koreatech_notice.py#L144 에 의해 계속 적재되고 있을거라 지우면 안될 듯

Comment on lines +10 to +11
`koin`.`dept_nums`,
`koin`.`dept_infos`,
Copy link
Contributor

Choose a reason for hiding this comment

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

얘네는 사용 중일 것 같은데?

@johnny19991006 johnny19991006 marked this pull request as draft March 14, 2024 02:40
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