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

[BE] chore: db replica 설정 제거 #994

Closed
wants to merge 1 commit into from

Conversation

skylar1220
Copy link
Contributor


🚀 어떤 기능을 구현했나요 ?

  • db replica 관련 코드를 제거했습니다.

🔥 어떻게 해결했나요 ?

  • datasource 패키지를 삭제했어요.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

develop에 쌓아뒀다가 나중에 한번에 release에 머지하면 prod.yml 설정과 불일치로 오류가 날 것이기때문에
해당 pr 어프로브 완료되면 바로 아래의 작업을 하려고합니다~

  1. secret prod.yml에서 데이터 소스 하나만 남기기
  2. 해당 pr 바로 hotfix로 release에 머지 + cd

📚 참고 자료, 할 말

@skylar1220 skylar1220 self-assigned this Dec 5, 2024
@skylar1220 skylar1220 changed the title chore: db replica 설정 제거 [BE] chore: db replica 설정 제거 Dec 5, 2024
Copy link

github-actions bot commented Dec 5, 2024

Test Results

157 tests   154 ✅  4s ⏱️
 58 suites    3 💤
 58 files      0 ❌

Results for commit 9cefc52.

@skylar1220
Copy link
Contributor Author

아래와 같은 이유로 close합니다~

  1. 인프라 변경(복원) 가능성을 중요하게 생각한다면 유지하는 것이 나음. 딘, 안쓰는 코드가 남고, 프록시를 거치는 것 감안해야함.

  2. 이미 애플리케이션에서는 인프라 상황에 따라서 의존적이지 않게 추상화 되어있음. 애플리케이션은 인프라의 구체적인 세부사항(DB 개수 등)을 알 필요가 없다. 추후 다시 DB 복제 환경으로 되돌아 간다고 해도 설정파일만 수정하면 동작 가능하다. 단일 DB 환경으로 통합했어도 애플리케이션은 여전히 읽기/쓰기 분리 환경을 지원하는 구조는 유연성 측면에서 적절하다고 생각된다.

  3. 사용하지 않게된 다른 의존성은 제거한 것과 비교해서 남겨두는 이유: 비교적 replica가 다시 도입될 가능성이 꽤 있겠다고 생각합니다. (TPS며 성능 확인할 때 replica를 하고 안하고가 꽤 컸던 걸 기억해보면 조금만 규모가 커져도 성능에 큰 영향을 미침)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] db replication 관련 코드를 제거한다
2 participants