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

[fix] Hikari Connection Pool 고갈 문제 해결 #135

Merged
merged 12 commits into from
Jan 15, 2024

Conversation

yonghwankim-dev
Copy link
Member

@yonghwankim-dev yonghwankim-dev commented Jan 12, 2024

구현한 것

  • 스케줄링 메소드의 cron 표현식 오타 수정
  • application.yml 파일에 OSIV(Open Session in View) 비활성화를 추가하였습니다.
    • sse 요청과 같은 http가 연결되는 동안 서비스 레이어의 트랜잭션이 종료되었음에도 �서비스 레이어 밖에서 유지되는 DB Connection Pool을 유지하지 않도록 하였습니다.
  • HasAuthorizationAspect AOP 컴포넌트에서 포트폴리오를 조회한다음에 member 엔티티 객체를 지연 로딩하는 방식에서 PortfolioService.hasAuthorizationBy 메소드로 이동시켜서 검증하는 방식으로 변경하였습니다. 변경 이유는 OSIV 비활성화에 따른 트랜잭션이 적용된 서비스 레이어의 메소드가 종료된 이후에 member 엔티티 객체를 지연 로딩 방식으로 호출하였기 때문입니다.

@yonghwankim-dev yonghwankim-dev added the bug Something isn't working label Jan 12, 2024
@yonghwankim-dev yonghwankim-dev self-assigned this Jan 12, 2024
@yonghwankim-dev yonghwankim-dev linked an issue Jan 12, 2024 that may be closed by this pull request
@yonghwankim-dev yonghwankim-dev merged commit 6401b79 into bug/#131-token Jan 15, 2024
@yonghwankim-dev yonghwankim-dev deleted the bug/#123-hikari branch January 15, 2024 05:10
yonghwankim-dev added a commit that referenced this pull request Jan 15, 2024
* #131 fix: 액세스 토큰 발급 오류 수정

* #131 test: 액세스 토큰 발급 테스트 코드 추가

* #131 test: 오타 수정

* [fix] Hikari Connection Pool 고갈 문제 해결 (#135)

* #123 docs: 테스트 프로파일에 히카리 로깅 레벨 설정

* #129 refactor: 별도 메소드 분리

* #129 test: 히카리 연결 풀 테스트 코드 추가

* #123 fix: 스케줄링 메소드 오타 수정

* #123 fix: throwable 로깅

* #131 test: HasPortfolioAuthorizationAspect 테스트 코드 추가

* #123 fix: OSIV(open session in view) 설정

* #123 test: 불필요한 테스트 코드 제거

* #123 refactor: 포트폴리오 조회시 fetch join으로 변경

- N+1 문제를 해결하기 위해서 fetch join 사용

* #123 test: findAllByPortfolioId 테스트 코드 추가

* #123 fix: OSIV 비활성화에 따른 fetch join 적용

* #123 fix: OSIV 비활성화에 따른 테스트 코드 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug] Hikari Connection Pool 고갈 문제
2 participants