feat: 쿼리 카운터 적용

[parkmuhyeun]

📄 Summary

• close: [BE] 쿼리 카운터 적용 #444

상세한 내용은 issue 혹은 블로그 참고하시면 될 것 같고 결과는 다음과 같습니다!

🙋🏻 More

[wonyongChoi05]

구웃~ 👍🏻 코어 로직이 아닌것치고 이정도면 충분한 퀄리티인 것 같네요 할 일을 만들어서 해주는 민무우~

[iamjooon2]

민무 idea 설정 바뀐듯

[parkmuhyeun]

import 패키지 이렇게 분리되는게 맞지 않나

[kyY00n]

수고했어 무민!!! 늦게 확인해서 미안합니당 ㅜㅜ 질문을 몇개 달았어요!
덕분에 공부를 하게 되어서 감사합니다.

[kyY00n]

리뷰하며 공부하기
메서드를 실행하기 전에 쿼리 카운트를 한답.

[kyY00n]

리뷰하며 공부하기

따라서 RequestContextHolder.getRequestAttributes()가 null을 반환하는 경우는 웹 요청이나 웹 관련 작업이 아닌 상황에서 발생합니다. 예를 들어, 백그라운드 작업, 스케줄러, 혹은 일반적인 Java 응용 프로그램 실행 중에는 RequestAttributes가 존재하지 않기 때문에 null을 반환합니다.

오오! 그래서 isRequest()라는 이름이 붙었구나.

[kyY00n]

한 가지.. 제가 잘 모르는 상태에서 봐서 아쉬웠던 점이 있는데요..!

countQuery 메서드로 분리되어서, invoke 메서드만 봤을 때는
이 핸들러가 무조건 쿼리를 센다는 것처럼 인식돼서 전체 구조를 보며 이해할 때 좀 헷갈리는 요인이 되었던 것 같습니당.
그런 면에서 메서드 분리를 하지 않아도 괜찮지 않았을까? 라는 생각이 들었어요 ☀️

이상 InvocationHandler 처음 본 인간의 하소연이었습니다.
반영은 무민 맘대로 해주시면 될 것 같아요. 구현해준 무민 최고~!

public Object invoke(Object proxy, Method method, Object[] args) throws Throwable {
    if (isPrepareStatement(method) && isRequest()) { // 이럴때만
        queryCounter.increaseCount(); // 쿼리를 세는 녀석입니당 ~
    }
    return method.invoke(connection, args);
}

[parkmuhyeun]

method.invoke가 원래 메서드가 발동하는 곳이고 countQuery는 부가적으로 넣은 것이라 후에 부가적으로 몇개 들어가게된다면 분리해주는 게 좋을 것 같아서 분리시켜놨습니다! 지금은 사실 딱히 분리 안 시켜줘도 상관은 없긴한데 고치면 approve가 다 풀릴거 같아서 이대로 가겠습니당

[kyY00n]

리뷰하며 공부하기
Connection에 적용된것은 프록시.
AOP는 DataSource.getConnection에만 적용한 것이다!

[kyY00n]

WARN_QUERY_COUNT가 8인 이유가 궁금합니다요! 🤓

[parkmuhyeun]

딱히 큰 이유는 없고, 5번과 10번중 고민하다 사이 값으로 했습니다!