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

INTERNAL: Close Ehcache CacheManager when shutdown FrontCacheMemcachedClient. #817

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

uhm0311
Copy link
Collaborator

@uhm0311 uhm0311 commented Sep 26, 2024

🔗 Related Issue

⌨️ What I did

  • FrontCacheMemcachedClient를 더 이상 사용하지 않게 된 경우, Ehcache3의 CacheManager 객체에 할당된 리소스를 즉시 해제합니다.

@jhpark816
Copy link
Collaborator

@cheesecrust @oliviarla
ehcache3 사용 방식에 일부 miss가 있었네요.
코드 확인하고 리뷰 의견 주세요.

cheesecrust
cheesecrust previously approved these changes Sep 27, 2024
@jhpark816
Copy link
Collaborator

@uhm0311 rebase 바랍니다.

@uhm0311
Copy link
Collaborator Author

uhm0311 commented Sep 27, 2024

@jhpark816

rebase 했습니다.

@jhpark816 jhpark816 merged commit f6e6a08 into naver:develop Sep 27, 2024
3 checks passed
@uhm0311 uhm0311 deleted the uhm0311/ref2 branch November 20, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants