Impl likeCount and isLiked Logic #79

davin111 · 2022-11-30T00:07:08Z

related #67, #78

강의평 리스트를 전달할 때 내가 해당 강의평을 공감했는지 나타내는 is_liked를 필드를 추가
여러 명이 동시에 like_count update 할 때, lost update 문제가 발생할 수 있는 부분을 optimistic lock 으로 방지
GET /v1/tags/main/{id}/evaluations에서 cache hit 된 강의평 리스트 전달 시, is_liked 와 like_count 는 따로 DB 에서 가져와 덮어쓰도록 로직 추가
동일 유저가 동일 강의평에 대해 공감/공감 취소한 경우, 200 -> 409 로 스펙 변경 cc. @woohm402

davin111 · 2022-11-30T00:42:09Z

.../main/kotlin/com/wafflestudio/snuttev/core/domain/evaluation/dto/EvaluationWithLectureDto.kt

@@ -13,12 +12,9 @@ data class EvaluationWithLectureDto @QueryProjection constructor(
    val lifeBalance: Double,
    val rating: Double,
    val likeCount: Long,
-    @get:JsonProperty("isHidden")
-    @field:JsonProperty("isHidden")


FasterXML/jackson-module-kotlin#80 이런 이슈 때문에 넣었던 건데 예전에 업데이트 하면서 필요 없어짐.

davin111 · 2022-11-30T00:45:58Z

core/src/main/kotlin/com/wafflestudio/snuttev/core/domain/evaluation/model/LectureEvaluation.kt

+@DynamicUpdate
+@OptimisticLocking(type = OptimisticLockType.DIRTY)


@Version을 위한 칼럼 따로 만드는 게 번거롭다고 여겨져서 이렇게 했습니다. 의견 주셔도 좋습니다.

요것도 괜찮네요

davin111 · 2022-11-30T00:47:33Z

...src/main/kotlin/com/wafflestudio/snuttev/core/domain/evaluation/service/EvaluationService.kt

+            postHitProcessor = { dtos ->
+                val evaluationsIds = dtos.map { it.id }
+                val likes = evaluationLikeRepository.findAllByLectureEvaluationIdIn(evaluationsIds)
+                dtos.map { dto ->
+                    dto.copy(
+                        likeCount = likes.count { it.lectureEvaluation.id == dto.id }.toLong(),
+                        isLiked = likes.any { it.lectureEvaluation.id == dto.id && it.userId == userId },
+                    )
+                }
+            },


cache hit 해놓고도 DB 에 접근하는 게 슬프지만, 일단 너무 많은 걸 당장 바꾸지 않고 기능이 완벽하게 동작하는 것에 집중했습니다. single thread 인 Redis 없이 DB 로만 해결하는 방법도 일부러 해보고 싶었고요.

더 좋은 방식에 대한 의견은 남겨놔주세요. ex> like_count 를 Redis 로 관리하자 등. (그런다면 is_liked 도 그렇게 해야할까?)

like 관련 로직은 새로 서비스를 하나 파서 분리하면 좋을 것 같습니다.

제 개인적인 의견으로는 둘 다 인메모리 캐싱을 해도 될 거 같아요.

해당 캐시 키는 user id 와 evaluation id 로 생각하시나요?

유저가 공감/공감 취소할 때마다 캐시 날리구요?

Redis 가 아닌 인 메모리로 보시는 이유도 궁금합니다.

인메모리가 더 빠르고, api 스펙 자체가 리소스 부담을 줄이기 위해 캐시 size를 작게 잡아도 miss가 많이 생길것 같지 않아서요.

PFCJeong · 2022-12-01T00:47:17Z

...om/wafflestudio/snuttev/core/domain/evaluation/repository/LectureEvaluationRepositoryImpl.kt


-    private fun JPAQueryFactory.selectEvaluationWithLectureDto() = select(
+    private fun JPAQueryFactory.selectEvaluationWithLectureDto(userId: String) = select(


쿼리가 많이 빡세네요.. 추후에 개선 포인트로 두면 좋을 것 같습니다

ㅋㅋ 좋습니다. 쿼리가 빡세다고 판단하신 이유는 join 3개 붙어서인가요?

넵. 근데 무엇보다 where 절에서 tag 구하려고 avg 들어가는게 있는데 풀스캔이 돌 것 같아서 요것만 잘 분리해도 좋을 것 같아용.

select le.id as col_0_0_, le.user_id as col_1_0_, le.content as col_2_0_, le.grade_satisfaction as col_3_0_, le.teaching_skill as col_4_0_, le.gains as col_5_0_, le.life_balance as col_6_0_, le.rating as col_7_0_, le.like_count as col_8_0_, le.is_hidden as col_9_0_, le.is_reported as col_10_0_, IF(elike.id is null, false, true) as col_11_0_, le.from_snuev as col_12_0_, sl.year as col_13_0_, sl.semester as col_14_0_, sl.lecture_id as col_15_0_, lecture.title as col_16_0_, lecture.instructor as col_17_0_ from lecture_evaluation le inner join semester_lecture sl on le.semester_lecture_id = sl.id left outer join evaluation_like elike on le.id = elike.lecture_evaluation_id and elike.user_id = '61f55ab5b7a1c400103aed78' inner join lecture on sl.lecture_id = lecture.id where ( sl.lecture_id in (select sl_tmp.lecture_id from lecture_evaluation le_tmp inner join semester_lecture sl_tmp on le_tmp.semester_lecture_id = sl_tmp.id where le_tmp.is_hidden = false group by sl_tmp.lecture_id having avg(le_tmp.rating) >= 4.0) ) and sl.classification = '교양' and le.is_hidden = false order by le.id desc limit 20;

참고로 현재 쿼리... ㅋㅋㅋ

where subquery 로 실행되는 부분을 따로 query 해서 page size (20) 개수만 가져온 다음,
in 이용한 query 하도록,
application 에서 이렇게 2개로 분리하는 식을 생각하신 건가요? (즉, in에 들어가는 개수를 줄일 수 있어서..?)

avg() 쓰는 이상 기본적으로 lectureEvaluation 대상으로 풀 스캔 도는 거 자체는 어쩔 수가 없지 않나 궁금해서 질문입니다.

@PFCJeong 질문이 많아 죄송해요! 요거 시간 나실 때 답해주시면 추후 PR 에 참고하겠습니다. (요것도 #79 (comment)?)

칼럼을 추가해서 강의평이 추가, 삭제, 수정될 때마다 미리 통계를 내서 태그 달아두면 좋을 것 같았어요.. 대대적인 공사가 필요하겠지만..

아하 ㅋㅋㅋ 비정규화해서 저장해두는 거네요. 그렇게 하면 정말 대공사가 필요하긴 하겠군요.

PFCJeong · 2022-12-01T00:47:33Z

core/src/main/kotlin/com/wafflestudio/snuttev/core/domain/evaluation/model/LectureEvaluation.kt

+@DynamicUpdate
+@OptimisticLocking(type = OptimisticLockType.DIRTY)


요것도 괜찮네요

PFCJeong · 2022-12-01T00:49:44Z

...src/main/kotlin/com/wafflestudio/snuttev/core/domain/evaluation/service/EvaluationService.kt

+            postHitProcessor = { dtos ->
+                val evaluationsIds = dtos.map { it.id }
+                val likes = evaluationLikeRepository.findAllByLectureEvaluationIdIn(evaluationsIds)
+                dtos.map { dto ->
+                    dto.copy(
+                        likeCount = likes.count { it.lectureEvaluation.id == dto.id }.toLong(),
+                        isLiked = likes.any { it.lectureEvaluation.id == dto.id && it.userId == userId },
+                    )
+                }
+            },


like 관련 로직은 새로 서비스를 하나 파서 분리하면 좋을 것 같습니다.

제 개인적인 의견으로는 둘 다 인메모리 캐싱을 해도 될 거 같아요.

subeenpark-io

큰 이견은 없는데,

동일 유저가 동일 강의평에 대해 공감/공감 취소한 경우, 200 -> 409 로 스펙 변경

요거 이유가 뭔가요?

davin111 · 2022-12-01T15:04:48Z

@subeenpark-io 프론트에서 그렇게 원했습니다. https://wafflestudio.slack.com/archives/C0PAVPS5T/p1669728281989599?thread_ts=1668299933.853369&channel=C0PAVPS5T&message_ts=1669728281.989599 cc. @woohm402

davin111 added 14 commits November 13, 2022 16:04

use querydsl and impl cache for evaluationservice

f165a86

cachekey ttl

e28180b

disable circular dep

5341910

remove empty lines

4112e6d

prod cache ttl 1h

10c3712

set up redis for github actions

2a75323

minor changes for cursor

c499deb

tmp

3d0ddf0

resolve conflicts

fe79c23

like

d1f3b94

fix typo

aa35b70

rm space ++ --

61b443d

Merge remote-tracking branch 'origin/develop' into feature/likes

7c6cba7

use optimistic lock and postHitProcessor

51e0c94

davin111 requested a review from PFCJeong November 30, 2022 00:07

davin111 requested review from Hank-Choi and subeenpark-io as code owners November 30, 2022 00:07

davin111 self-assigned this Nov 30, 2022

davin111 requested a review from woohm402 November 30, 2022 00:08

modify test

6c9d311

davin111 commented Nov 30, 2022

View reviewed changes

resolve conflicts

71848e6

PFCJeong reviewed Dec 1, 2022

View reviewed changes

subeenpark-io approved these changes Dec 1, 2022

View reviewed changes

davin111 merged commit 4e3f5fc into develop Dec 1, 2022

davin111 deleted the feature/likes branch December 1, 2022 15:05

davin111 mentioned this pull request Apr 13, 2023

강의평 수정 API 개발 #86

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Impl likeCount and isLiked Logic #79

Impl likeCount and isLiked Logic #79

davin111 commented Nov 30, 2022 •

edited

Loading

davin111 Nov 30, 2022

davin111 Nov 30, 2022

PFCJeong Dec 1, 2022

davin111 Nov 30, 2022 •

edited

Loading

PFCJeong Dec 1, 2022

davin111 Dec 1, 2022

PFCJeong Dec 12, 2022

PFCJeong Dec 1, 2022

davin111 Dec 1, 2022

PFCJeong Dec 1, 2022

davin111 Dec 1, 2022

davin111 Dec 11, 2022

PFCJeong Dec 12, 2022

davin111 Dec 18, 2022

PFCJeong Dec 1, 2022

PFCJeong Dec 1, 2022

subeenpark-io left a comment •

edited

Loading

davin111 commented Dec 1, 2022

		@DynamicUpdate
		@OptimisticLocking(type = OptimisticLockType.DIRTY)


		private fun JPAQueryFactory.selectEvaluationWithLectureDto() = select(
		private fun JPAQueryFactory.selectEvaluationWithLectureDto(userId: String) = select(

Impl likeCount and isLiked Logic #79

Impl likeCount and isLiked Logic #79

Conversation

davin111 commented Nov 30, 2022 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

davin111 Nov 30, 2022 • 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

subeenpark-io left a comment • edited Loading

Choose a reason for hiding this comment

davin111 commented Dec 1, 2022

davin111 commented Nov 30, 2022 •

edited

Loading

davin111 Nov 30, 2022 •

edited

Loading

subeenpark-io left a comment •

edited

Loading