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.

Sign up for GitHub

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

Jump to bottom

Refactor: 루트 도메인 문자열 캐싱 전략을 변경한다. #104

Merged
merged 5 commits into from
Apr 28, 2024
Merged

Refactor: 루트 도메인 문자열 캐싱 전략을 변경한다. #104

merged 5 commits into from
Apr 28, 2024

Conversation

hseong3243
Copy link
Owner

작업 내용

  • 루트 도메인 문자열 캐싱 쓰기 전략에 write through를 적용했습니다. DomainRepositoryImpl의 save() 메서드에 캐시 insert 로직을 추가하였습니다.
  • 루트 도메인 문자열 캐시 워밍업을 추가하였습니다. ApplicationReadyEvent를 바라보는 이벤트 리스너를 추가하여 애플리케이션 시작시 캐시 워밍업 동작이 일어납니다.
  • Trie 구현을 개선했습니다. 기존에는 insert() 메서드에 synchronized 키워드를 이용해 동시성을 처리하였으나, Tire의 Node가 사용하는 Map 구현을 ConcurrentHashMap으로 변경하고 이를 잘 활용할 수 있도록 로직을 개선했습니다.

@hseong3243 hseong3243 added the Refactor 프로젝트 구조 및 코드를 개선합니다. label Apr 28, 2024
@hseong3243 hseong3243 self-assigned this Apr 28, 2024
@hseong3243 hseong3243 linked an issue Apr 28, 2024 that may be closed by this pull request
Refactor: 루트 도메인 문자열 캐싱 전략을 변경한다. #101
Closed
@github-actions GitHub Actions
Copy link

Test Results

200 tests  +1   200 ✅ +1   3s ⏱️ ±0s
 59 suites +2     0 💤 ±0 
 59 files   +2     0 ❌ ±0 

Results for commit 98f34a7. ± Comparison against base commit 7c0a59e.

@hseong3243 hseong3243 merged commit b98f084 into main Apr 28, 2024
2 checks passed
@hseong3243 hseong3243 deleted the refactor/#101-change-trie-strategy branch April 28, 2024 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@hseong3243 hseong3243

Labels
Refactor 프로젝트 구조 및 코드를 개선합니다.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Refactor: 루트 도메인 문자열 캐싱 전략을 변경한다.
1 participant
@hseong3243