refactor: 팔로잉 소식 조회 쿼리 성능 개선

module-infrastructure/persistence-database/src/main/java/com/depromeet/followinglog/repository/FollowingMemoryLogRepository.java

@@ -1,14 +1,13 @@
 package com.depromeet.followinglog.repository;
 
+import static com.depromeet.followinglog.entity.QFollowingMemoryLogEntity.followingMemoryLogEntity;
+import static com.depromeet.friend.entity.QFriendEntity.friendEntity;
 import static com.depromeet.member.entity.QMemberEntity.memberEntity;
 import static com.depromeet.memory.entity.QMemoryEntity.memoryEntity;
-import static com.depromeet.memory.entity.QStrokeEntity.strokeEntity;
 
 import com.depromeet.followinglog.domain.FollowingMemoryLog;
 import com.depromeet.followinglog.entity.FollowingMemoryLogEntity;
-import com.depromeet.followinglog.entity.QFollowingMemoryLogEntity;
 import com.depromeet.followinglog.port.out.persistence.FollowingMemoryLogPersistencePort;
-import com.depromeet.friend.entity.QFriendEntity;
 import com.querydsl.core.types.dsl.BooleanExpression;
 import com.querydsl.jpa.impl.JPAQueryFactory;
 import java.util.List;
@@ -21,9 +20,6 @@ public class FollowingMemoryLogRepository implements FollowingMemoryLogPersisten
     private final JPAQueryFactory queryFactory;
     private final FollowingMemoryLogJpaRepository followingMemoryLogJpaRepository;
 
-    private QFollowingMemoryLogEntity followingMemoryLog =
-            QFollowingMemoryLogEntity.followingMemoryLogEntity;
-
     @Override
     public Long save(FollowingMemoryLog followingMemoryLog) {
         return followingMemoryLogJpaRepository
@@ -33,66 +29,53 @@ public Long save(FollowingMemoryLog followingMemoryLog) {
 
     @Override
     public List<FollowingMemoryLog> findLogsByMemberIdAndCursorId(Long memberId, Long cursorId) {
-        QFriendEntity friend = QFriendEntity.friendEntity;
-
         List<FollowingMemoryLogEntity> contents =
                 queryFactory
-                        .selectFrom(followingMemoryLog)
-                        .join(followingMemoryLog.memory, memoryEntity)
-                        .fetchJoin()
-                        .join(followingMemoryLog.memory.member, memberEntity)
+                        .selectFrom(followingMemoryLogEntity)
+                        .join(followingMemoryLogEntity.memory, memoryEntity)
                         .fetchJoin()
-                        .leftJoin(followingMemoryLog.memory.memoryDetail)
+                        .join(memoryEntity.member, memberEntity)
                         .fetchJoin()
-                        .leftJoin(followingMemoryLog.memory.strokes, strokeEntity)
+                        .leftJoin(memoryEntity.memoryDetail)
                         .fetchJoin()
-                        .join(friend)
-                        .on(friend.following.eq(memberEntity))
+                        .join(friendEntity)
+                        .on(friendEntity.following.eq(memberEntity))
                         .fetchJoin()
-                        .where(friend.member.id.eq(memberId), cursorIdLt(cursorId))
+                        .where(friendEntity.member.id.eq(memberId), cursorIdLt(cursorId))
                         .limit(11)
-                        .orderBy(followingMemoryLog.id.desc())
+                        .orderBy(followingMemoryLogEntity.id.desc())
                         .fetch();
-        queryFactory
-                .selectFrom(followingMemoryLog)
-                .join(followingMemoryLog.memory, memoryEntity)
-                .fetchJoin()
-                .join(followingMemoryLog.memory.member, memberEntity)
-                .fetchJoin()
-                .leftJoin(followingMemoryLog.memory.images)
-                .fetchJoin()
-                .fetch();
         return contents.stream().map(FollowingMemoryLogEntity::toModel).toList();
     }
 
     @Override
     public void deleteAllByMemoryIds(List<Long> memoryIds) {
-        queryFactory.delete(followingMemoryLog).where(memoryIdIn(memoryIds)).execute();
+        queryFactory.delete(followingMemoryLogEntity).where(memoryIdIn(memoryIds)).execute();
     }
 
     @Override
     public void deleteAllByMemoryId(Long memoryId) {
-        queryFactory.delete(followingMemoryLog).where(memoryIdEq(memoryId)).execute();
+        queryFactory.delete(followingMemoryLogEntity).where(memoryIdEq(memoryId)).execute();
     }
 
     private BooleanExpression memoryIdIn(List<Long> memoryIds) {
         if (memoryIds == null) {
             return null;
         }
-        return followingMemoryLog.memory.id.in(memoryIds);
+        return followingMemoryLogEntity.memory.id.in(memoryIds);
     }
 
     private BooleanExpression memoryIdEq(Long memoryId) {
         if (memoryId == null) {
             return null;
         }
-        return followingMemoryLog.memory.id.eq(memoryId);
+        return followingMemoryLogEntity.memory.id.eq(memoryId);
     }
 
     private BooleanExpression cursorIdLt(Long cursorId) {
         if (cursorId == null) {
             return null;
         }
-        return followingMemoryLog.id.lt(cursorId);
+        return followingMemoryLogEntity.id.lt(cursorId);
     }
 }

module-presentation/src/main/java/com/depromeet/friend/facade/FollowFacade.java

@@ -45,7 +45,9 @@ public boolean addOrDeleteFollow(Long memberId, FollowRequest followRequest) {
         }
 
         boolean isAdd = followUseCase.addOrDeleteFollow(member, following);
-        eventPublisher.publishEvent(FollowLogEvent.of(following, member));
+        if (isAdd) {
+            eventPublisher.publishEvent(FollowLogEvent.of(following, member));
+        }
 
         return isAdd;
     }