[BE] TicketRepository, StageRepository의 JPQL을 수정한다.

[seokjin8678]

✨ 세부 내용

현재 다음과 같이 /festivals/1, /stages/1/tickets로 요청을 보낼 때 다음과 같이 티켓의 정렬 순서가 같지 않고 다르게 나옵니다.

원인은 다음과 같습니다.

TicketRepository

findAllDetailByFestivalId() 메소드에 JPQL이 잘못되었습니다.

다음과 같은 JPQL 코드를

@Query("""
        SELECT s FROM Stage s
        LEFT JOIN FETCH s.tickets t
        LEFT JOIN FETCH t.ticketAmount
        WHERE s.festival.id = :festivalId
        """)
List<Stage> findAllDetailByFestivalId(@Param("festivalId") Long festivalId);

다음과 같이 변경하면 됩니다.

@Query("""
        SELECT s FROM Stage s
        LEFT JOIN FETCH s.tickets t
        JOIN FETCH t.ticketAmount
        WHERE s.festival.id = :festivalId
        """)
List<Stage> findAllDetailByFestivalId(@Param("festivalId") Long festivalId);

Ticket과 TicketAmount는 같은 생명 주기를 가집니다.
따라서 Ticket이 존재하면 TicketAmount는 무조건 존재할 수 밖에 없고 유일하기 때문에 LEFT JOIN을 할 필요가 없습니다.
정확한 원인은 모르겠으나, LEFT JOIN을 할 때 순서가 바뀌는 것 같습니다...

LEFT JOIN을 사용한 쿼리의 EXPLAIN

id	select_type	table	partitions	type	possible_keys	key	key_len	ref	rows	filtered	Extra
1	SIMPLE	s1_0	null	ref	fk_stage__festival	fk_stage__festival	9	const	1	100	null
1	SIMPLE	t1_0	null	ALL	fk_ticket__stage	null	null	null	2	100	Using where; Using join buffer (hash join)
1	SIMPLE	t2_0	null	ALL	PRIMARY	null	null	null	2	100	Using where; Using join buffer (hash join)

JOIN을 사용한 쿼리의 EXPLAIN

id	select_type	table	partitions	type	possible_keys	key	key_len	ref	rows	filtered	Extra
1	SIMPLE	s1_0	null	ref	PRIMARY,fk_stage__festival	fk_stage__festival	9	const	1	100	null
1	SIMPLE	t2_0	null	ALL	PRIMARY	null	null	null	2	100	Using join buffer (hash join)
1	SIMPLE	t1_0	null	ALL	PRIMARY,fk_ticket__stage	null	null	null	2	50	Using where; Using join buffer (hash join)

또한 StageRepository에 N+1 문제가 발생하는 것이 파악되었고 해당 메서드를 Fetch Join을 사용하여 N+1 문제를 해결할 수 있습니다.

⏰ 예상 소요 시간

30분

[seokjin8678]

TicketAmount의 LEFT JOIN을 생략하니 Ticket이 없는 Stage 조회가 안되는 문제가 생기네요 😂
왜냐하면 TicketAmount를 JOIN 한다면 TicketAmount가 무조건 있어야 한다는 소리이고, 따라서 Ticket이 없는 Stage가 조회가 안됩니다. 😂😂😂
순서는 다음과 같이 뒤에 ORDER BY를 명시하면 해결할 수 있기는 한데...
안드로이드 측에서 ticketType으로 정렬을 한다면 굳이 해결을 하지 않아도 될 문제 같습니다.