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] EntityManager close 추가 #89

Merged
merged 2 commits into from
Aug 11, 2023
Merged

Conversation

jun02160
Copy link
Member

@jun02160 jun02160 commented Aug 11, 2023

📌 관련 이슈

close #88

✨ 어떤 이유로 변경된 내용인지

  • 스케줄링 작업에서 EntityManager를 이용해 수동으로 merge를 한 후, close 하는 부분이 없어 커넥션 풀에 누수가 발생한 것으로 예측됨
    -> 일단 em.close() 를 추가해서 해결

  • application.yml에서는 hikari 옵션 추가

🙏 검토 혹은 리뷰어에게 남기고 싶은 말

아직 스케줄링으로 푸시알림이 제대로 구현되는지 테스트는 안 된 상태입니다 .. 추가적으로 해결하고 다시 이 부분 테스트 해볼게요 !

@jun02160 jun02160 added 🔥 Pull Request PR 날림 🚨Hotfix 급한 수정사항 발생 예준🍒 🐞BugFix 에러 수정, 버그 수정 labels Aug 11, 2023
@jun02160 jun02160 requested a review from ddongseop August 11, 2023 18:11
@jun02160 jun02160 self-assigned this Aug 11, 2023
Copy link
Member

@ddongseop ddongseop left a comment

Choose a reason for hiding this comment

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

잘 됐 으면 좋 겠 다~

@jun02160 jun02160 merged commit 0d29204 into develop Aug 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
예준🍒 🐞BugFix 에러 수정, 버그 수정 🚨Hotfix 급한 수정사항 발생 🔥 Pull Request PR 날림
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] Hikari Connection Pool 에러 해결
2 participants