#564 [feat] sql 로깅 모듈 구현 #568
Conversation
| if (tmpsql.startsWith("create") || tmpsql.startsWith("alter") || tmpsql.startsWith("comment")) { | ||
| sql = FormatStyle.DDL.getFormatter().format(sql); | ||
| } else { | ||
| sql = FormatStyle.BASIC.getFormatter().format(sql); | ||
| } |
There was a problem hiding this comment.
DDL과 일반 쿼리를 나눠 포맷팅 해서 더 좋은것 같습니다!
| public void sqlLoggingPoint() { | ||
| } | ||
|
|
||
| @Around("sqlLoggingPoint()") |
There was a problem hiding this comment.
요 around로 쿼리 실행될 때 시작지점 기록을 하는거군요 👍
| MDC.put(MDC_KEY, "[ QUERY START -> " + joinPoint.getSignature().toShortString() + "]"); | ||
|
|
||
| return joinPoint.proceed(); | ||
| } |
There was a problem hiding this comment.
P5) 찾아보니 mdc는 스레드로컬 방식으로 관리되는데 aop로 sql 실행시점에 mdc 에 데이터를 넣고 그대로 두면 그 스레드가 이후 다른작업을 할때도 같은 값을 사용할 수 있어서 메서드가 끝나면 mdc.remove나 clear를 하는 게 좋다고 합니다! try-finally 사용해서 메서드 실행 후, finally { MDC.remove(MDC_KEY); } 해주는건 어떻게 생각하시나요??
There was a problem hiding this comment.
옷 아주 좋습니다! request ID 담고 있는 부분도 포함해서 수정하겠습니다!!
There was a problem hiding this comment.
오호라 request Id 부분에는 제가 이미 mdc.clear()을 구현해놨네요! clear()을 통해 로컬 스레드 맵을 지우는 방식으로 진행하겠습니다!
There was a problem hiding this comment.
실제로 디버깅 해보니 Interceptor에 clear해주기 때문에 AOP에서도 clear하진 않아도 될 것 같습니다! clear해주는 ThreadLocalMap이 모두 같은 객체더라고요! 같이 알면 좋을 것 같아 남깁니다! 앞에 인터셉터에서 해주길 잘했네요 ㅎㅎ
✒️ 관련 이슈번호
Key Changes 🔑
결과는 아래와 같습니다!
To Reviewers 📢