[22기_서가영] 성능최적화 미션 제출합니다.

[caminobelllo]

성능 최적화 대상

1. SeatQueryService

• getSeatMap 메서드에서 상영 정보(screening) - 쿼리 1, 상영관의 모든 좌석(seats) - 쿼리 2, 해당 상영의 예매된 좌석(reservedSeat) - 쿼리3 을 조회하고 리턴 시에 Java 메모리를 사용
• 쿼리 2, 3은 동일 관심사인데도 각각 불러와서 처리함 -> 메모리에 모두 올려 비교

[ 최적화 내용 ]

• LEFT JOIN을 사용해 앞서 쿼리 2의 정보를 기준으로 쿼리 3에 대한 정보를 더함
• SeatRepository에서 SeatStatusDto를 직접 반환하는 JPQL 쿼리를 추가함
• DB에서 조인 계산을 하기 때문에 서비스 로직이 단순해짐

2. CinemaService.getCinema(Long cinemaId) - N+1 문제

Cinema cinema = cinemaRepository.findByIdWithAuditoriums(cinemaId)...
List<AuditoriumDto> auditoriums = cinema.getAuditoriums()
        .stream().map(AuditoriumDto::from).toList();

public static AuditoriumDto from(Auditorium auditorium) {
    return new AuditoriumDto(
            auditorium.getName(),
            auditorium.getDescription(),
            auditorium.getAuditoriumType().getIsSpecial() 
    );
}

• findByIdWithAuditoriums에서 쿼리 1번
• stream().map() 실행 시 AuditoriumDto.from()이 상영관 개수(N개)만큼 호출됨
• from 내부에서 auditorium.getAuditoriumType()을 호출 -> 여기서 AutitoriumType을 LAZY로 설정해놨기 때문에 이시점에 N개 추가 쿼리 발생

[ 최적화 내용 ]

• CinemaRepository의 findByIdWithAuditoriums 메소드에 @entitygraph를 추가해서
• 연관된 엔티티의 연관된 엔티티까지 한 번에 가져오도록 함

3. ProductOrderQueryService.getOrder

• 주문 정보 조회 (쿼리 1), 주문에 포함된 아이템 목록 (쿼리 2), 아이템 목록을 dto로 변환 시 (getProduct().getName() 호출, getProduct().getPrice() 호출 ==> 여기 각각에서 N+1 쿼리 발생)
• 총 쿼리 발생 횟수 2+N

[ 최적화 내용 ]

• ProductOrderItem을 조회할 때, 연관된 Product 엔티티를 JOIN FETCH로 한 번에 가져오도록 JPQL을 수정
• 그래서 앞에 쿼리2 부분에서 ProductOrderItem과 Product 정보가 모두 로드되도록 함

[danaggero]

고생 많으셨습니다! db 인덱스에 대해서 잘 정리해주셔서 잘 배웠습니다!

[danaggero]

정리 너무 잘해주셨네요!