N + 1 문제 발견

[Combi153]

소위 N + 1 문제라 일컬어지는 상황을 마주한 것 같아 자료 남기고 공유합니다.
다같이 해결해보면 좋겠습니다.

API 와 쿼리

특정 여행의 모든 감상을 조회하는 API 를 실행했을 때 쿼리는 아래 순서로 실행됩니다.

• GET /trips/{tripId}/posts
• 쿼리 개수 : 14
• 시간 : 195ms

SELECT COUNT(*) FROM member WHERE member_id=?

select 
    member_id, 
    created_at, 
    nickname, 
    oauth_id, 
    oauth_type, 
    updated_at
from 
    member 
where 
    member_id = ?

select
    trip_id,
    created_at,
    image_url,
    member_id,
    name,
    route_image_url,
    status,
    updated_at
from 
    trip 
where 
    trip_id = ?

SELECT
    post_id,
    address,
    created_at,
    member_id,
    point_id,
    post_image_url,
    route_image_url,
    title,
    trip_id,
    updated_at,
    writing
FROM
    post
WHERE
    trip_id=?

이후 아래 쿼리가 10번 실행됩니다.

select 
    p.point_id,
    p.created_at,
    p.has_post,
    p.latitude,
    p.longitude,
    p.recorded_at,
    t.trip_id,
    t.created_at,
    t.image_url,
    t.member_id,
    t.name,
    t.route_image_url,
    t.status,
    t.updated_at,
    p.updated_at
from 
    point p
left join 
    trip t on t.trip_id = p.trip_id
where 
    p.point_id = ?

이때 실제 조회되는 감상 데이터도 10개 입니다.

DB schema

code

• 엔티티

@Entity
public class Post extends BaseEntity {

    @Id
    @GeneratedValue(strategy = IDENTITY)
    @Column(name = "post_id")
    private Long id;

    @OneToOne
    @JoinColumn(name = "point_id")
    private Point point;

    // (...)
}

@Entity
public class Point extends BaseEntity {

    @Id
    @GeneratedValue(strategy = IDENTITY)
    @Column(name = "point_id")
    private Long id;
		
    // (...)
}

• 서비스 메서드

@Transactional
@Service
public class PostService {
		
    // (...)

    @Transactional(readOnly = true)
    public PostsResponse readAllByTripId(LoginUser loginUser, Long tripId) {
        Member member = memberRepository.getById(loginUser.memberId());
        Trip trip = tripRepository.getById(tripId);
        trip.validateAuthorization(member);

        return postRepository.findAllByTripId(tripId).stream()
                .sorted(comparing(Post::pointRecordedAt).reversed())
                .collect(collectingAndThen(toList(), PostsResponse::from));
    }
}

[greeng00se]

공유 감사합니다 같이 발견했을 때의 기쁨을 아직 잊지 못하겠어요

[Jaeyoung22]

Fetch Join, Lazy Loading으로 해결했습니다