[BE] 비밀번호 마이그레이션 로직 삭제 #386
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ikjo39
requested review from
ehBeak,
hw0603,
seokmyungham and
seunghye218
as code owners
Test Results153 tests 153 ✅ 20s ⏱️ Results for commit 6715eb8. ♻️ This comment has been updated with latest results. |
ehBeak
approved these changes
Oct 6, 2024
There was a problem hiding this comment.
고생 많으셨습니다. 다온😊
말씀하신 대로, 원래 의도했던 부분은 쿼리가 실행하는 시점마다 해당 쿼리가 replica로 가는지, source로 가는지 확인하려고 하셨지만, 현재는 AbstractRoutingDataSource가 스레드 변경 시점마다 로그를 찍어주기 때문에 의도치 않게 여러 번 찍게 되는군요!
그래서 로그 수준을 운영환경에 남기지 않기 위해 debug로 옮겼고요.
사소하지만, 원래 처음 의도하셨던 방법을 고민해 보는 것도 좋을 것 같아요.
운영 환경에서 쿼리가 나갈 때, 어떤 DB로 갔는지 로그로 표시해 두면 이후 문제가 생겼을 때 로그를 확인해서 해결하기 좋을 것 같아서요 😊
지금 당장은 크게 중요한 것이 아니기 때문에 이후에 생각해 보면 좋을 것 같습니다! (저도 같이 고민해 볼게요~~)
seunghye218
approved these changes
Oct 9, 2024
hw0603
approved these changes
Oct 10, 2024
ikjo39
force-pushed
the
hotfix/376-delete-unuse-code
branch
from
5188a60 to
6715eb8
Compare
ehBeak
pushed a commit
that referenced
this pull request
Oct 11, 2024
* chore: 서브모듈 변경사항 반영 * hotfix: 불필요한 메서드 삭제 * refactor: 로그 레벨 debug로 수정
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
관련 이슈
작업 내용
평문화 비밀번호 마이그레이션 코드 삭제
비밀번호 마이그레이션을 완료하여 이젠 사용하지 않는 비밀번호 마이그레이션 코드를 삭제하였습니다.
AbstractRoutingDataSource 내 로그 레벨 수정
현재 Datasource는 쿼리가 발생하는 시점에 Source DB와 Replica DB를 동적으로 선택하는데요.
이때 연결된 Datasource가 무엇인지 알고자 로그로 남겨두었는데요.
AbstractRoutingDataSource가 스레드가 변경되어 데이터베이스 작업을 시작할 때마다 어떤 데이터소스를 사용할지 결정하기 위해 determineCurrentLookupKey()를 호출하게 되어 의도한 동작과 다를 때도 로그를 찍히는 것을 발견하게 되었습니다.불필요하게 많아지는 로그를 방지하기 위해 debug 레벨로 로그를 남기도록 수정하였습니다 :)