[Feat/#221] 아티클 조회수 저장 방식 및 조회 방식 개선 #223

hun-ca · 2024-07-20T06:44:29Z

🎫 연관 이슈

resolved #221

💁‍♂️ PR 내용

flyway 'V1.00.0.14' 배포: ARTICLE_VIEW_COUNT 테이블 추가
조회수를 가져오는 방식
- as-is: 조회 요청이 왔을때 즉시 count 쿼리
- to-be: ARTICLE_VIEW_COUNT에서 article id로 조회수 자체를 select
ARTICLE_VIEW_COUNT 테이블은 추후 메인 페이지 내 아티클을 조회수 기반으로 정렬할 때에도 사용될 중요한 테이블로서 역할을 할 예정입니다

🙏 작업

🙈 PR 참고 사항

해당 PR 배포 후 아래 SQL롤 테스트 확인 필요

SELECT * FROM ARTICLE_VIEW_COUNT ORDER BY view_count DESC LIMIT 10;

📸 스크린샷

🤖 테스트 체크리스트

체크 미완료
체크 완료

…ount -> select)

hun-ca

리뷰 부탁드립니다

hun-ca · 2024-07-20T06:45:31Z

data/db/migration/entity/V1.00.0.14__manage_article_view_count.sql

+CREATE TABLE ARTICLE_VIEW_COUNT
+(
+    article_id BIGINT NOT NULL,
+    view_count BIGINT NOT NULL,
+    deleted_at TIMESTAMP NULL DEFAULT NULL,
+    CONSTRAINT article_view_count_pk PRIMARY KEY (article_id)
+);


ARTICLE_VIEW_COUNT 테이블 정의

(참고로 PK 이름 지정함: article_view_count_pk)

hun-ca · 2024-07-20T06:46:40Z

api-repo/src/main/kotlin/com/few/api/repo/dao/article/ArticleViewCountDao.kt

+    fun upsertArticleViewCount(query: ArticleViewCountQuery) {
+        dslContext.insertInto(ARTICLE_VIEW_COUNT)
+            .set(ARTICLE_VIEW_COUNT.ARTICLE_ID, query.articleId)
+            .set(ARTICLE_VIEW_COUNT.VIEW_COUNT, 1)
+            .onDuplicateKeyUpdate()
+            .set(ARTICLE_VIEW_COUNT.VIEW_COUNT, ARTICLE_VIEW_COUNT.VIEW_COUNT.plus(1))
+    }


ARTICLE_VIEW_COUNT 테이블에 조회수 컬럼을 upsert함.
기존 로우가 존재하면 count 값만 늘리고 로우 자체가 없으면 Insert진행 (최초 아티클 조회시에 해당)

hun-ca · 2024-07-20T06:47:58Z

api/src/main/kotlin/com/few/api/domain/article/handler/ArticleViewHisAsyncHandler.kt

+
+            articleViewCountDao.upsertArticleViewCount(ArticleViewCountQuery(articleId))
+            log.debug { "Successfully upserted article view count for articleId: $articleId and memberId: $memberId" }


기존에 아티클 view his 테이블에 저장하는걸 async로 구성했는데, 마찬가지로 해당 과정에서 Insert가 끝나면 ARTICLE_VIEW_COUNT 테이블에 Upsert를 진행함. (어제 DB 트리거로 만들어뒀던 부분을 애플리케이션 단으로 옮긴 것)

hun-ca · 2024-07-20T06:50:03Z

api/src/main/kotlin/com/few/api/domain/article/handler/ArticleViewCountHandler.kt

+@Component
+class ArticleViewCountHandler(
+    private val articleViewCountDao: ArticleViewCountDao,
+) {
+    @Transactional(isolation = Isolation.READ_UNCOMMITTED, propagation = Propagation.REQUIRES_NEW)
+    fun browseArticleViewCount(articleId: Long): Long {
+        return (articleViewCountDao.selectArticleViewCount(ArticleViewCountCommand(articleId)) ?: 0L) + 1L
+    }
+}


아티클 조회수를 조회하는 서비스

Isolation.READ_UNCOMMITTED: 조회수 데이터 정합성 필요없이 최대한 빠르게 조회하기 위해 격리 레벨을 최하위로 낮춤. 조회수 자체가 크게 중요한 데이터도 아니고 해서 커밋 안되었더라도 읽게 함

Propagation.REQUIRES_NEW: 기존 트랜잭션이 있을 경우 새로운 트랜잭션을 만들어서 진행. (기존 트랜잭션에 격리레벨을 분리하기 위함)

hun-ca · 2024-07-20T06:54:40Z

data/db/migration/entity/V1.00.0.14__manage_article_view_count.sql

+-- 조회수 순으로 아티클 조회시 사용하기 위한 인덱스
+-- ex. SELECT * FROM ARTICLE_VIEW_COUNT ORDER BY view_count;
+CREATE INDEX article_view_count_desc_idx1 ON ARTICLE_VIEW_COUNT(view_count DESC);


메인 페이지에서 아티클 전체 리스트업 정렬 기준이 조회수로 내림차순 되기 때문에 필요한 인덱스 사전작업

ex SQL

SELECT * FROM ARTICLE_VIEW_COUNT ORDER BY view_count DESC limit 50;

hun-ca

변경 사항 재확인 부탁드려요...

hun-ca · 2024-07-20T12:08:11Z

data/db/migration/entity/V1.00.0.14__manage_article_view_count.sql

+CREATE TABLE ARTICLE_VIEW_COUNT
+(
+    article_id  BIGINT  NOT NULL,
+    view_count  BIGINT  NOT NULL,
+    category_cd TINYINT NOT NULL,
+    deleted_at  TIMESTAMP NULL DEFAULT NULL,
+    CONSTRAINT article_view_count_pk PRIMARY KEY (article_id)
+);


ARTICLE_VIEW_COUNT 테이블 CATEGORY_CD 컬럼 추가 반영. 해당 값은 아티클 마스터에 있는 컬럼이지만 정규화하면 오히려 손해임

hun-ca · 2024-07-20T12:09:13Z

api-repo/src/main/kotlin/com/few/api/repo/dao/article/ArticleViewCountDao.kt

+    fun upsertArticleViewCount(query: ArticleViewCountQuery) {
+        dslContext.insertInto(ARTICLE_VIEW_COUNT)
+            .set(ARTICLE_VIEW_COUNT.ARTICLE_ID, query.articleId)
+            .set(ARTICLE_VIEW_COUNT.VIEW_COUNT, 1)
+            .set(ARTICLE_VIEW_COUNT.CATEGORY_CD, query.categoryType?.code)
+            .onDuplicateKeyUpdate()
+            .set(ARTICLE_VIEW_COUNT.VIEW_COUNT, ARTICLE_VIEW_COUNT.VIEW_COUNT.plus(1))
+    }


ARTICLE_VIEW_COUNT 테이블 upsert 쿼리 수정: CATEGORY_CD 추가 반영. 메인화면 아티클 조회수 별 정렬에서 사용될 테이블인데, 정렬기준에 카테고리 까지는 포함이 되어야 할 거 같음...

upsert 쿼리가 무엇인지 알 수 있나요?

테이블에 업데이트할 로우가 없으면 인서트하고, 있으면 업데이트

hun-ca · 2024-07-20T12:09:30Z

data/db/migration/entity/V1.00.0.14__manage_article_view_count.sql

+CREATE INDEX article_view_count_idx1 ON ARTICLE_VIEW_COUNT (view_count DESC);
+
+-- 카테고리 별 필터링을 위한 인덱스
+CREATE INDEX article_view_count_idx2 ON ARTICLE_VIEW_COUNT (category_cd);


카테고리 코드에 인덱싱 추가

특별히 카테고리를 넣은 이유는 메인 페이지에서 카테고리별 정렬될때 사용됨... article_view_count 테이블 자체가 거기서 메인페이지에서 아티클 ordering할 기준 테이블이 되기 때문에 여기에 카테고리가 있어야지 없으면 ordering할 떄 다시 아티클 테이블 가서 조회해 와야 함

belljun3395 · 2024-07-21T11:41:49Z

api-repo/src/main/kotlin/com/few/api/repo/dao/article/ArticleViewCountDao.kt

+            .set(ARTICLE_VIEW_COUNT.ARTICLE_ID, query.articleId)
+            .set(ARTICLE_VIEW_COUNT.VIEW_COUNT, 1)
+            .set(ARTICLE_VIEW_COUNT.CATEGORY_CD, query.categoryType?.code)
+            .onDuplicateKeyUpdate()


요거는 뭔가요?

article_view_count 테이블 로우는 아티클 당 한개로 구성되는데, 첫번째로 조회될 땐 insert가 가고, 이미 row가 있는 경우(이미 1번 이상 조회됨) Update(view_count + 1) 쿼리가 날라가는 upsert 쿼리입니다

아하 신기하다..!!!

SQL 변환

INSERT INTO ARTICLE_VIEW_COUNT (ARTICLE_ID, VIEW_COUNT, CATEGORY_CD) VALUES ($articleId, 1, $category) ON DUPLICATE KEY UPDATE VIEW_COUNT = VIEW_COUNT + 1;

belljun3395 · 2024-07-21T11:42:40Z

api/src/main/kotlin/com/few/api/domain/article/handler/ArticleViewCountHandler.kt

+class ArticleViewCountHandler(
+    private val articleViewCountDao: ArticleViewCountDao,
+) {
+    @Transactional(isolation = Isolation.READ_UNCOMMITTED, propagation = Propagation.REQUIRES_NEW)


요거 @ViewTransactional 요런 어노테이션으로 선언 해버릴까요?

belljun3395 · 2024-07-21T11:44:42Z

api/src/main/kotlin/com/few/api/domain/article/usecase/ReadArticleUseCase.kt

-
-        articleViewHisAsyncHandler.addArticleViewHis(useCaseIn.articleId, useCaseIn.memberId)
+        // ARTICLE VIEW HIS에 저장하기 전에 먼저 VIEW COUNT 조회하는 순서 변경 금지
+        val views = articleViewCountHandler.browseArticleViewCount(useCaseIn.articleId)


요친구는 왜 네이밍이 handler이죠? 저는 반환을 하니 service가 어울릴꺼 같아요

service도 생각했었는데 같은 도메인이여서 네이밍 뭘로할지 고민되네요

진짜 애매하다..

belljun3395 · 2024-07-21T12:08:49Z

api/src/main/kotlin/com/few/api/domain/article/handler/ArticleViewHisAsyncHandler.kt

 ) {
    private val log = KotlinLogging.logger {}

    @Async(value = DATABASE_ACCESS_POOL)
    @Transactional
-    fun addArticleViewHis(articleId: Long, memberId: Long) {
+    fun addArticleViewHis(articleId: Long, memberId: Long, categoryType: CategoryType?) {


요기 ? 를 사용하기 전에 처리하고 들어가는게 좋을 것 같아요!

UC에서 카테고리 가져올때 CategoryType 자체적으로 코드 -> 객체 변환시 nullable 리턴이라 저길 어떻게 바꾸면 좋을지…

요청 전에 ?: 을 사용해서 확인하고 널이면 예외를 발생시키면 될 것같아요

…hunca

hun-ca · 2024-07-21T12:54:10Z

api/src/main/kotlin/com/few/api/domain/article/usecase/ReadArticleUseCase.kt

-            CategoryType.fromCode(articleRecord.category)
+            CategoryType.fromCode(articleRecord.category) ?: throw RuntimeException("invalid article")


모든 아티클은 카테고리가 필수인데, 카테고리에 정의되지 않은 경우가 발생하는거 자체가 말이 일단 안되는 상황이라.. 예외코드를 정의하고 던지는게 맞을까 싶어요

저도 그렇긴 한데 말이 안되는 상황도 대비하는 것은 나쁘지 않은 것 같아요

hun-ca · 2024-07-21T13:02:52Z

api/src/main/resources/messages/article.properties

@@ -1,2 +1,3 @@
 article.notfound.id=\u0061\u0072\u0074\u0069\u0063\u006c\u0065\u002e\u006e\u006f\u0074\u0066\u006f\u0075\u006e\u0064\u002e\u0069\u0064
 article.notfound.articleidworkbookid=\u0061\u0072\u0074\u0069\u0063\u006c\u0065\u002e\u006e\u006f\u0074\u0066\u006f\u0075\u006e\u0064\u002e\u0061\u0072\u0074\u0069\u0063\u006c\u0065\u0069\u0064\u0077\u006f\u0072\u006b\u0062\u006f\u006f\u006b\u0069
+article.invalid.category=\uc874\uc7ac\ud558\uc9c0\u0020\uc54a\ub294\u0020\uc544\ud2f0\ud074\u0020\uce74\ud14c\uace0\ub9ac\uc785\ub2c8\ub2e4\u000d


article.invalid.category 에러 코드 추가 (모든 아티클은 카테고리가 있기 때문에 이게 발생하는 상황이 오면 안됨)

유니코드 변환: https://www.learnkoreantools.com/kr/korean-to-unicode

hun-ca added 5 commits July 20, 2024 15:08

feat: ARTICLE_VIEW_COUNT 테이블 생성, 아티클 조회수 순 정렬 등에 사용 예정

1c16093

fix: ARTICLE_VIEW_COUNT 테이블 deleted_at 추가

d8acd58

feat: 아티클 조회시 article view his가 아닌 article view count를 보도록 수정(SL수정: c…

6fca14c

…ount -> select)

test: ReadArticleUseCaseTest 수정 반영

5e41a3d

feat: ArticleViewCountHandler 추가

13306e6

hun-ca added refactor 기존 기능에 대해 개선할 때 사용됩니다. DB DB 관련 수정이 이러날 때 사용됩니다. labels Jul 20, 2024

hun-ca self-assigned this Jul 20, 2024

hun-ca requested a review from belljun3395 as a code owner July 20, 2024 06:44

github-actions bot added the feature 새로운 기능을 만들 때 사용됩니다 label Jul 20, 2024

fix: view_count 컬럼 기준 내림차순 정렬 인덱스로 변경

3f6a688

hun-ca commented Jul 20, 2024

View reviewed changes

hun-ca added 2 commits July 20, 2024 21:04

feat: category_cd 컬럼 추가 in ARTICLE_VIEW_COUNT 테이블 and 인덱싱

b5f58ce

test: category_cd 컬럼 추가 테스트 반영

a1eb90a

hun-ca commented Jul 20, 2024

View reviewed changes

belljun3395 approved these changes Jul 21, 2024

View reviewed changes

hun-ca and others added 2 commits July 21, 2024 20:54

Merge remote-tracking branch 'origin/main' into feat/#221_hunca

dc4a03a

Merge branch 'main' into feat/#221_hunca

0ff043e

belljun3395 reviewed Jul 21, 2024

View reviewed changes

hun-ca added 2 commits July 21, 2024 21:46

Merge remote-tracking branch 'origin/feat/#221_hunca' into feat/#221_…

bc4d0ac

…hunca

refactor: article category 유효하지 않을 경우에 대한 검증 추가

680cf75

hun-ca commented Jul 21, 2024

View reviewed changes

feat: article.invalid.category 에러코드 추가

c519e60

hun-ca commented Jul 21, 2024

View reviewed changes

hun-ca merged commit 28d00e8 into main Jul 21, 2024
6 checks passed

This was referenced Jul 24, 2024

articles 목록 조회 API 신규 개발 #228

Closed

[Feat/#228] 아티클 목록 조회(무한스크롤) API 추가 #259

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat/#221] 아티클 조회수 저장 방식 및 조회 방식 개선 #223

[Feat/#221] 아티클 조회수 저장 방식 및 조회 방식 개선 #223

hun-ca commented Jul 20, 2024 •

edited

Loading

hun-ca left a comment

hun-ca Jul 20, 2024

hun-ca Jul 20, 2024

hun-ca Jul 20, 2024

hun-ca Jul 20, 2024

hun-ca Jul 20, 2024

hun-ca Jul 20, 2024

hun-ca left a comment

hun-ca Jul 20, 2024

hun-ca Jul 20, 2024

belljun3395 Jul 21, 2024

hun-ca Jul 21, 2024

hun-ca Jul 20, 2024

hun-ca Jul 21, 2024

belljun3395 Jul 21, 2024

hun-ca Jul 21, 2024

belljun3395 Jul 21, 2024

hun-ca Jul 21, 2024

belljun3395 Jul 21, 2024

belljun3395 Jul 21, 2024

hun-ca Jul 21, 2024

belljun3395 Jul 21, 2024

belljun3395 Jul 21, 2024

hun-ca Jul 21, 2024

belljun3395 Jul 21, 2024

hun-ca Jul 21, 2024

belljun3395 Jul 21, 2024

hun-ca Jul 21, 2024

hun-ca Jul 21, 2024


		articleViewCountDao.upsertArticleViewCount(ArticleViewCountQuery(articleId))
		log.debug { "Successfully upserted article view count for articleId: $articleId and memberId: $memberId" }

		CategoryType.fromCode(articleRecord.category)
		CategoryType.fromCode(articleRecord.category) ?: throw RuntimeException("invalid article")

[Feat/#221] 아티클 조회수 저장 방식 및 조회 방식 개선 #223

[Feat/#221] 아티클 조회수 저장 방식 및 조회 방식 개선 #223

Conversation

hun-ca commented Jul 20, 2024 • edited Loading

🎫 연관 이슈

💁‍♂️ PR 내용

🙏 작업

🙈 PR 참고 사항

📸 스크린샷

🤖 테스트 체크리스트

hun-ca 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hun-ca 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

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

SQL 변환

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

hun-ca commented Jul 20, 2024 •

edited

Loading