[Feat/#238] 데이터 베이스 케시 설정 #246

belljun3395 · 2024-07-23T12:26:52Z

🎫 연관 이슈

resolved: #238

💁‍♂️ PR 내용

워크북 데이터 베이스 케시 설정

🙏 작업

SELECT_WORKBOOK_RECORD_CACHE 추가
SELECT_WRITER_CACHE 추가
selectWorkBook 메서드 캐시 설정
selectWriter 메서드, selectWriters 메서드 데이터 캐시 설정
캐싱된 데이터 활용을 위한 XXXCacheManager 구현

🙈 PR 참고 사항

📸 스크린샷

selectWorkBook 캐시 확인

엔트리 사이즈 3으로 설정후 아이디 1 -> 2 -> 3 -> 4 -> 4 조회 결과

selectWriter 메서드, selectWriters 메서드 캐시 확인

아티클 조회시 selectWriter 수행됨
워크북 조회시 selectWriters 수행됨

워크북 1에 포함된 아티클 조회

selectWriter에서 캐시 저장

워크북 1 조회

selectWriters에서 캐시에 저장된 값 사용

🤖 테스트 체크리스트

체크 미완료
체크 완료

hun-ca

고생하셨습니다! 추가적으로 캐시 라이브러리를 호출하는 MemberCacheManager 코드에서 예외가 발생될 부분이 있는지 검토 부탁드립니다. 캐시 접근에 대한 예외가 API 실패로 연결되는건 바람직하지 않을것 같아서요

hun-ca · 2024-07-24T11:48:56Z

api-repo/src/main/kotlin/com/few/api/repo/config/LocalCacheConfig.kt

+            Eh107Configuration.fromEhcacheCacheConfiguration(cache10ConfigurationBuilder)
+        val selectWorkBookRecordCacheConfig: javax.cache.configuration.Configuration<Any, Any> =
+            Eh107Configuration.fromEhcacheCacheConfiguration(cache10ConfigurationBuilder)


캐시 컨피그 객체는 공유해도 잘 동작하나요?

네 런타임에 컨피그 객체를 수정하는 것이 없어 문제 없을 것 같습니다.

hun-ca · 2024-07-24T11:59:55Z

api-repo/src/main/kotlin/com/few/api/repo/dao/member/MemberDao.kt

+            .where(Member.MEMBER.ID.`in`(notCachedId))
            .and(Member.MEMBER.TYPE_CD.eq(MemberType.WRITER.code))


.where(Member.MEMBER.ID.in(notCachedId)) 에서 충분히 작가가 필터링 될 수 있기 때문에 .and(Member.MEMBER.TYPE_CD.eq(MemberType.WRITER.code)) 는 생략해도 될 것 같아요. 안그래도 TYPE_CD에 인덱싱도 없을거 같구요

selectWritersQueryExplain.txt

실행계획을 보면 Member.MEMBER.TYPE_CD.eq(MemberType.WRITER.code) 가 영향이 없어 보이는데 맞을까요..?

row가 적어서 실행계획이 크게 의미가 없긴 한데, 만약 로우가 엄청 많아질 때를 가정하면 TYPE_CD에 인덱스가 없어 필터 조건에 넣을지 고민해봐야 할거 같아요. 지금은 일단 이상없을거고 나중에 실행계획 보고 생각해보시죠

hun-ca · 2024-07-24T12:01:44Z

api-repo/src/main/kotlin/com/few/api/repo/dao/member/MemberDao.kt

+        val cachedId = cachedValues.map { it.writerId }
+        val notCachedId = query.writerIds.filter { it !in cachedId }


별건 아니지만 cachedId, notCachedId 변수는 복수형으로 바꾸는게 좋을거 같습니다. 딱 봤을때 컬렉션 타입인지 알 수 있어서요

520898a 에서 수정하였습니다.

hun-ca · 2024-07-24T12:03:57Z

api-repo/src/main/kotlin/com/few/api/repo/dao/member/MemberCacheManager.kt

+    private val cacheManager: CacheManager,
+) {
+
+    private var selectWriterCache: Cache<Any, Any> = cacheManager.getCache(SELECT_WRITER_CACHE)?.nativeCache as Cache<Any, Any>


호오옥시나 런타임 중에 cacheManager.getCache(SELECT_WRITER_CACHE)?.nativeCache 레퍼런스가 바뀌진 않겠죠?

디컴파일 해보면 CacheMamager 생성 시점에 위의 코드가 실행되어서 문제 없을 것 같아요

오.. 굳잡

belljun3395

리뷰 감사합니다!

belljun3395 · 2024-07-25T04:54:11Z

api-repo/src/main/kotlin/com/few/api/repo/config/LocalCacheConfig.kt

+            Eh107Configuration.fromEhcacheCacheConfiguration(cache10ConfigurationBuilder)
+        val selectWorkBookRecordCacheConfig: javax.cache.configuration.Configuration<Any, Any> =
+            Eh107Configuration.fromEhcacheCacheConfiguration(cache10ConfigurationBuilder)


네 런타임에 컨피그 객체를 수정하는 것이 없어 문제 없을 것 같습니다.

belljun3395 · 2024-07-25T04:56:14Z

api-repo/src/main/kotlin/com/few/api/repo/dao/member/MemberCacheManager.kt

+    private val cacheManager: CacheManager,
+) {
+
+    private var selectWriterCache: Cache<Any, Any> = cacheManager.getCache(SELECT_WRITER_CACHE)?.nativeCache as Cache<Any, Any>


디컴파일 해보면 CacheMamager 생성 시점에 위의 코드가 실행되어서 문제 없을 것 같아요

belljun3395 · 2024-07-25T04:57:55Z

api-repo/src/main/kotlin/com/few/api/repo/dao/member/MemberDao.kt

+        val cachedId = cachedValues.map { it.writerId }
+        val notCachedId = query.writerIds.filter { it !in cachedId }


520898a 에서 수정하였습니다.

belljun3395 · 2024-07-25T05:02:26Z

api-repo/src/main/kotlin/com/few/api/repo/dao/member/MemberDao.kt

+            .where(Member.MEMBER.ID.`in`(notCachedId))
            .and(Member.MEMBER.TYPE_CD.eq(MemberType.WRITER.code))


selectWritersQueryExplain.txt

실행계획을 보면 Member.MEMBER.TYPE_CD.eq(MemberType.WRITER.code) 가 영향이 없어 보이는데 맞을까요..?

hun-ca and others added 4 commits July 22, 2024 11:01

[Fix/#233] addArticleViewHis에서 runCatching 후 예외 발생시 트랜잭션 롤백되도록 수정

dd1d3ad

feat: SELECT_WORKBOOK_RECORD_CACHE 추가

32771bc

refactor: selectWorkBook 메서드 데이터 베이스 케시 설정

7acd8a8

feat: WorkBookCacheManager 구현

62dcbca

github-actions bot assigned belljun3395 Jul 23, 2024

github-actions bot added config 설정 파일과 관련된 내용을 다룰 때 사용됩니다 feature 새로운 기능을 만들 때 사용됩니다 labels Jul 23, 2024

belljun3395 added 3 commits July 23, 2024 22:11

feat: SELECT_WRITER_CACHE 추가

8c03f66

feat: MemberCacheManager 구현

510dd53

feat: MemberCacheManager 적용

a402340

belljun3395 requested a review from hun-ca July 23, 2024 13:26

belljun3395 added 2 commits July 24, 2024 00:34

refactor: 동시성을 고려한 수정

7007a35

refactor: 캐시 사이즈 수정

41b540b

hun-ca approved these changes Jul 24, 2024

View reviewed changes

belljun3395 added 2 commits July 25, 2024 13:53

refactor: 변수 이름 수정

bde32d0

refactor: 복수형 이름으로 수정

520898a

belljun3395 commented Jul 25, 2024

View reviewed changes

belljun3395 force-pushed the main branch 2 times, most recently from dd1d3ad to 4d5a4bd Compare July 26, 2024 07:29

belljun3395 force-pushed the feat/#238_belljun3395 branch from d38780d to 520898a Compare July 26, 2024 13:24

Merge branch 'main' into feat/#238_belljun3395

eb3e1e9

belljun3395 merged commit b319fd0 into main Jul 28, 2024
5 checks passed

belljun3395 changed the title ~~[Feat/#238] 워크북 데이터 베이스 케시 설정~~ [Feat/#238] 데이터 베이스 케시 설정 Jul 30, 2024

belljun3395 added the ⭐major⭐ 주요 이슈(추후 회고 및 복귀 시 참고용) label Aug 1, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat/#238] 데이터 베이스 케시 설정 #246

[Feat/#238] 데이터 베이스 케시 설정 #246

belljun3395 commented Jul 23, 2024 •

edited

Loading

hun-ca left a comment •

edited

Loading

hun-ca Jul 24, 2024

belljun3395 Jul 25, 2024

hun-ca Jul 24, 2024

belljun3395 Jul 25, 2024

hun-ca Jul 25, 2024

hun-ca Jul 24, 2024

belljun3395 Jul 25, 2024

hun-ca Jul 24, 2024

belljun3395 Jul 25, 2024

hun-ca Jul 25, 2024

belljun3395 left a comment

belljun3395 Jul 25, 2024

belljun3395 Jul 25, 2024

belljun3395 Jul 25, 2024

belljun3395 Jul 25, 2024

		.where(Member.MEMBER.ID.`in`(notCachedId))
		.and(Member.MEMBER.TYPE_CD.eq(MemberType.WRITER.code))

		val cachedId = cachedValues.map { it.writerId }
		val notCachedId = query.writerIds.filter { it !in cachedId }

[Feat/#238] 데이터 베이스 케시 설정 #246

[Feat/#238] 데이터 베이스 케시 설정 #246

Conversation

belljun3395 commented Jul 23, 2024 • edited Loading

🎫 연관 이슈

💁‍♂️ PR 내용

🙏 작업

🙈 PR 참고 사항

📸 스크린샷

selectWorkBook 캐시 확인

selectWriter 메서드, selectWriters 메서드 캐시 확인

워크북 1에 포함된 아티클 조회

워크북 1 조회

🤖 테스트 체크리스트

hun-ca left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

belljun3395 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

belljun3395 commented Jul 23, 2024 •

edited

Loading

hun-ca left a comment •

edited

Loading