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

DB Replication 적용 #724

Merged
merged 12 commits into from
Oct 19, 2023
Merged

DB Replication 적용 #724

merged 12 commits into from
Oct 19, 2023

Conversation

mcodnjs
Copy link
Collaborator

@mcodnjs mcodnjs commented Oct 18, 2023

📄 Summary

#712

으아아 이오 홍고와 함께한 길고 길었던 DB Replication 입니다.
문제가 끝난 줄 알았지만 아직 사소한 버그들이 있습니다.
우선 저희가 겪었던 문제는 p6spy로 인한 문제였고, 관련하여서는 조금 더 자세히 정리해서 업로드 하겠습니다!

현재 남은 문제로,
DB Replication을 위해 프록시 객체를 사용하면서 쿼리 카운터에 x3의 쿼리 개수를 증가시키는 마법이 발생하고 있습니다.
관련해서는 조금 더 찾아봐야 알겠지만, 이를 도입해서 나타나는 문제로 인지되고,
실제 쿼리 개수는 x3이 아닌 이전과 동일하게 나가는 것으로 판단하였습니다. (로그만 증가한 것, 아니면 말씀 부탁)

그리고, p6spy 같은 경우에는 운영 환경에서 성능 문제로 사용이 권장되지 않습니다.
(+ replication 적용을 위해서는 p6spy를 도입할 수 없습니다)
따라서, 현재 서브 모듈에서 default를 false로 해두고, local 및 test 환경에서만 사용 가능하도록 변경해놓았습니다.
현재 정확한 동작을 위해 p6spy 의존성을 완전히 제거하였습니다 ..
이 부분도 확인 부탁드리겠습니다. . 감사합니다 ..

🙋🏻 More

너무 어렵습니다

@mcodnjs mcodnjs linked an issue Oct 18, 2023 that may be closed by this pull request
1 task
@github-actions
Copy link

github-actions bot commented Oct 18, 2023

📝 Jacoco Test Coverage

Total Project Coverage 78.98%
File Coverage [82.12%]
CityService.java 100% 🍏
CategoryService.java 100% 🍏
TripService.java 97.67% 🍏
LedgerService.java 96.53% 🍏
ItemService.java 91.86% 🍏
SharedTripService.java 91.53% 🍏
CommunityService.java 81.38% 🍏
LoginService.java 26.54%
RoutingDataSource.java 0%
DataSourceType.java 0%
DataSourceConfig.java 0%

Copy link
Collaborator

@LJW25 LJW25 left a comment

Choose a reason for hiding this comment

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

트랜잭션의 마법..드디어 탈출..

@mcodnjs mcodnjs merged commit 7c44150 into develop Oct 19, 2023
1 check passed
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.

DB 복제 및 Spring boot 적용
4 participants