Skip to content

Commit b1b7865

Browse files
committed
fix: #236 resolve N+1 query issue with fetch join
1 parent 34d4f25 commit b1b7865

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

src/main/java/doldol_server/doldol/rollingPaper/repository/custom/MessageRepositoryCustomImpl.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,12 @@ public class MessageRepositoryCustomImpl implements MessageRepositoryCustom {
2121
public MessageResponse getMessage(Long messageId, Long userId) {
2222
QMessage message = QMessage.message;
2323
QUser fromUser = new QUser("fromUser");
24+
QUser toUser = new QUser("toUser");
2425

2526
Message result = queryFactory
2627
.selectFrom(message)
27-
.join(message.from, fromUser)
28+
.join(message.from, fromUser).fetchJoin()
29+
.join(message.to, toUser).fetchJoin()
2830
.where(
2931
message.id.eq(messageId),
3032
message.from.id.eq(userId),
@@ -38,7 +40,7 @@ public MessageResponse getMessage(Long messageId, Long userId) {
3840

3941
MessageType messageType = MessageType.SEND;
4042

41-
return MessageResponse.of(result, result.getName(), result.getTo().getName(), messageType);
43+
return MessageResponse.of(result, result.getFrom().getName(), result.getTo().getName(), messageType);
4244
}
4345

4446
@Override

0 commit comments

Comments
 (0)